-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[Bug]: NC 28.0.2 - Icons in toolbar are black instead of white #43298
Comments
As a workaround I added the following custom CSS:
I understand the intention of the original filter invented with NC 28.0.2 - but maybe it would be better to have some option for the icon color instead of a filter. Edit: another "fix" is to make the background color a bit darker in the personal settings, so the filter to change the icon color to black will not trigger. However this will only work individually for a single user account and the "active tab" pill is still wrong (always white even with black app icons). Having the possibility to define a global system wide theme including the icon colors (and the "active tab" pill of course) would be much better. |
I also have this issue. The fix @arnowelzel gave worked for the header icons, but in my case the favicons are also generated as black on blue and are not fixed. |
Did you try to clear the cache of your browser? In my case the favicons are ok, but I cleared the browser cache after adding the custom CSS |
I stand corrected - indeed some apps have black favicons, but not all of them. I don't understand, how this can happen at all. Even if the favicons are generated, the colors used for that should be based on global preferences and not depending on some app specific rules. Correct favicons: Wrong favicons: |
The color of the text is based on your background color, it is chosen to have at least 4.5:1 contrast, meaning if you make the background color a bit darker it will be white text. |
Changing the tone of blue solved it for me, thank you @susnux. I don't believe the behavior should have changed like this, but I guess that is another debate. |
Just updated to 29.0.0 and it is not fixed. Icons are still black. There is a filter applies, when shouldn't .app-menu-main .app-menu-entry img[data-v-62f74e6b] {
filter: var(--background-image-invert-if-bright);
} |
Bug description
All icons in the toolbar are black instead of white.
Steps to reproduce
Expected behavior
Icons in the tab bar are white and not black.
Installation method
Community Web installer on a VPS or web space
Nextcloud Server version
28
Operating system
Debian/Ubuntu
PHP engine version
PHP 8.3
Web server
Apache (supported)
Database engine version
MariaDB
Is this bug present after an update or on a fresh install?
Updated from a MINOR version (ex. 22.1 to 22.2)
Are you using the Nextcloud Server Encryption module?
None
What user-backends are you using?
Configuration report
No response
List of activated Apps
No response
Nextcloud Signing status
No response
Nextcloud Logs
No response
Additional info
The text was updated successfully, but these errors were encountered: