-
Notifications
You must be signed in to change notification settings - Fork 1
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
Make a favicon #148
Comments
@rlskoeser @jhimpele I have created a favicon for the site. The favicon is what you see in your tab on the browser like the below – so when someone is viewing the lenape site they will see a visual that represents the project. This is super small so it needs to be readable. I am proposing two versions, do you like any of these two proposals? Here is v1 and here is v2 |
Thanks for working on this, Gissoo. I like v1. |
Same for me: v1 |
Orange for Princeton could work, but I wonder if there is enough contrast at small scale. @gissoo what do you think? Otherwise I like using light green color we used for resistance & resurgence (that's the one you're using now, right?). |
@jhimpele @rlskoeser Thanks! Regarding using the orange, believe it or not it did cross my mind but I didn't feel comfortable with it. Here is the orange I can go either way, I can also think of ways to justify it. I would like to also know Keely's opinion about it.– your comments made me catch something :( – our current orange color is the only leaf color that's not passing the accessibility test on the brown background, we might need to revise it and go with this color – it will still look distinguishable from the other leaves |
@gissoo I wondered about the contrast when I looked at the favicon! Could you show us the revised color in context with the other leaves? |
It's possible that the orange favicon could seem university-centric, but we did adopt that reason for making the university the center and orange branch of the tree.....I don't have strong feelings about the favicon color.... |
@jhimpele do you have any thoughts on the revised orange color per my earlier comment above, or are you ok with it? |
I'm fine with using the orange leaf for our favicon too, I agree with you, I checked with Keely too and she is fine too. |
Excellent! |
Thanks for checking with Keely. All looks good. It's hard to tell, but the orange in the legend still looks darker....At any rate this new color works. Thanks for all the fast progress this week. |
@gissoo if the orange is not enough contrast then I think we should switch; we'll need all the variant colors for the new orange (if they should change), but it's easy enough to update the color variables in the stylesheets. I like both of them and don't feel strongly about it visually. I don't have a preference for the color for the favicon; orange is fine with me. |
@jhimpele thank you for flagging! I updated the new orange color on the legend as well. |
@gissoo that's great about the colors. Do you want to make the change? Here's the relevant part of the file on the develop branch: https://github.com/Princeton-CDH/lenape-timetree/blob/develop/themes/timetree/assets/scss/base/_colors.scss#L16-L30 (If it doesn't make sense to you or doesn't look easy, I can handle it easily enough - it looks like there may be some redundancy I can clean up anyway). Please don't rearrange the branch colors. I've already mapped the a-e sets of colors to the named branches in the order we decided on, so rearranging them now would be confusing and cause more work. |
@rlskoeser Thank you! I have made the change to the color, and generated the favicon files, detailed information regarding the location of the files and the code are in implementation issue #152 that I just created. I'll go ahead and close this issue |
No description provided.
The text was updated successfully, but these errors were encountered: