-
Notifications
You must be signed in to change notification settings - Fork 2
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 favicon #103
Comments
Thoughts on any of these @thekaveman ? Happy to iterate |
Ooooo these are fun! I like 1, 6, and 8 the most -- not sure if you can squeeze all 3 highlight colors in one of those? Maybe make the cross-hatch box purple in 8? |
Sounds good! Will also double-check how these look at the actual favicon size in the next round |
@segacy1 7 and 8 (especially with the additional purple suggestion above) look good. |
@segacy1 or maybe, at the risk of offending your design sensibilities, adding a small purple circle to the empty space on 8? |
Vyki commented that 3/7/11 are closest to our actual "brand mark" i.e. the one that came out of the design process from Sublation a few years ago. So I think we should probably stick to one of those but with the added pops of color over the straight black/white version. |
Out of those options, I like 3 the most. If possible, we should change the white to another color to work on both light/dark modes. For instance, I use light-mode and don't think white will show through. Unless @thekaveman, there is a way to switch the logo based on the user's light/dark-mode. |
It has been many months!! I'm pulling in 3rd for us to use because:
That said, they're all options are saved in Figma if you'd like to try! @AnthonyRollins because there is a black fill included, it shouldn't matter if you're in light or dark mode! |
New favicon is live on the website! If your browser is using a cached old favicon on https://compiler.la, try Ctrl + F5 to do a refresh without cached content |
@vykster would like a favicon inspired by the new homepage icons (with colors):
As Vyki says:
The text was updated successfully, but these errors were encountered: