-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Use dark icons for app management #649
Comments
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: Replace: |
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: server/core/templates/layout.user.php Lines 116 to 119 in 5d0f9da
Otherwise we would have some white icons on white background for apps we don't maintain. |
Agreed. I suggested that earlier more as a hack for if we didn't find a better way, which we did. |
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 😄 |
@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. 😉 |
As a follow-up of #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
The text was updated successfully, but these errors were encountered: