Closed jancborchardt closed 8 years ago
Let me know if you need the actual SVG's modified to make them darker (ie if it's not being done in code), and I'll take care of it. Given the simplicity of the icons though, I'd say there's a good chance we could just do a find and replace on them all for:
Find: fill="#FFF"
Replace: fill="#000"
I would like to take this up if no one is doing this. :-)
Simply replacing the fill works for me. There is a PR already on this. ;)
I'd propose to just keep with the white icons as svg files and use the color inversion trick (like here: https://github.com/nextcloud/server/blob/5d0f9dabf6ee6ca432e99ad8cb1170badfabfd62/core/templates/layout.user.php#L116-L119) to render them as black icons in the app management as well.
Otherwise we would have some white icons on white background for apps we don't maintain.
I'd propose to just keep with the white icons as svg files and use the color inversion trick
Agreed. I suggested that earlier more as a hack for if we didn't find a better way, which we did.
pr #834 stable10 #897
I would Like to use a dark color for the bar at top. How could I change to normal?
@Mannshoch you probably use the »Direct Menu« which conveniently enough is also developed by the awesome @juliushaertl ;)
@juliushaertl if that app is enabled, then for bright header bar colors it probably makes sense to use the dark icons just as we switch the font to black for better contrast.
@jancborchardt That should already work like that :smile:
@juliushaertl hehe! Nice :D
@Mannshoch what’s your problem then?
@jancborchardt Yes, your Right I use »Direct Menu« in which CSS File do I have to manipulate this? Or would you maybe add a checkbox which allow to switch the color?
@Mannshoch the color of the icons should be switched automatically if the chosen theming color is too bright. (If you are using direct_menu version 0.9.1 or later) Maybe opening an issue over at https://github.com/juliushaertl/direct_menu/ would be better then discussing here. :wink:
As a follow-up of https://github.com/nextcloud/server/pull/627 (white app menu with dark icons) it would be good for consistency if the app management would also use dark icons. Then we can also get rid of the strange grey box we needed to make the white icons visible.
@juliushaertl are you also up for this? ;) cc @nextcloud/designers