-
Notifications
You must be signed in to change notification settings - Fork 0
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
SVG / PNG icons #158
Comments
|
ok, please provide all the group image SVGs |
this is more complex then I thought, let's discuss next meeting if it's worth to invest the time for this |
We keep getting negative feedback on the icon colors. As we discussed, the new design has a light blue background on the homepage and a white bg on internal pages. This makes it very hard to find an icon color that will work on both. We discussed changing the color programmatically via CSS according to the context. Turns out this doesn't work when the SVG is an external file. Since this keeps coming up I looked into it again, seems like there's a hack that might work for us. Can you take a look? |
I couldn't get it to work, the example in the link you sent doesn't work for me. It seems that this feature is partially supported in most browsers, and also depends on how the SVG file is structured. |
Could it be a CORS issue? It seems CORS applies to SVG, and when I try it locally I get a CORS error |
Moving to TODO for now. |
The new design requires group icons to have a different color in the homepage and internal pages.
Possible solutions:
Find some color that'll work with both (likely require changing the homepage design)
Have two sets of PNG icons, one for each color
Use SVG icons so we change color via the CSS, eg a class for each case
Seems like 3 is the best option as it will offer us more versatility and will be easier to use when site maintainers add new icons. @OriHoch please take a look and see how much work is needed to implement.
(GH doesn't allow attaching SVG to an issue. For testing you can get svg/png files of our Religion icon here)
The text was updated successfully, but these errors were encountered: