Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update icons to Zotero 7 style #501

Open
adomasven opened this issue Aug 7, 2024 · 9 comments · May be fixed by #502
Open

Update icons to Zotero 7 style #501

adomasven opened this issue Aug 7, 2024 · 9 comments · May be fixed by #502

Comments

@adomasven
Copy link
Member

With Zotero 7 beta available, we can probably just do this immediately. That way we can also address #487

@dstillman
Copy link
Member

This would also be the time to consider badging the icon with a red Z, like people have been asking us to do for years.

@dstillman
Copy link
Member

(Though we can just swap in the new icons first just to get it done.)

@adomasven
Copy link
Member Author

This would also be the time to consider badging the icon with a red Z, like people have been asking us to do for years.

Like this?
image

@dstillman
Copy link
Member

Without the huge knockout, but yeah

@adomasven
Copy link
Member Author

That is the badge background. It can't be made transparent. Or the badge has to be baked into the icons, but that will not look good.

@dstillman
Copy link
Member

dstillman commented Aug 7, 2024

Oh, no, I meant we just badge the icons. This looks terrible, so this is not an option.

@dstillman
Copy link
Member

@yexingsha can take a look at baking it in.

@adomasven
Copy link
Member Author

We generate greyscale icons for safari (with imagemagick), because if not safari used to do it itself and poorly. Safari now allows colored icons in extensions. Shall we stop doing that?

@dstillman
Copy link
Member

Yes! I hate the Safari icons.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

2 participants