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

Change main logo #3746

Merged
merged 3 commits into from May 22, 2023
Merged

Conversation

yayuyokitano
Copy link
Member

A new logo is necessary for approval to safari.

Replaces the main logo across src and xcode project, and makes a lot of updates to icon build scripts.

One not directly related change is a change to icon creation functions that specifies widths and heights an extra time. The lack of this was causing scaling bugs.

macos icon next to safari icon
image

General safari icon
image

Universal icon for non-safari browsers (both firefox and chromium)
image

Bigger version:
image

Realistically would close #2430 if acceptable.

Does not change action icons yet, though this should probably be done.

@yayuyokitano yayuyokitano added core This issue or pull request is related to the extension core major-change For major changes labels May 21, 2023
@jaccarmac
Copy link
Contributor

Looks great to me (haven't tried building yet).

Copy link
Member

@inverse inverse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 🚀

@jaccarmac jaccarmac merged commit 472561c into web-scrobbler:master May 22, 2023
8 checks passed
@yayuyokitano yayuyokitano deleted the universalicons branch May 28, 2023 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core This issue or pull request is related to the extension core major-change For major changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New extension icon
3 participants