server icon indicating copy to clipboard operation
server copied to clipboard

[Bug]: Theme color not applied properly to icons

Open SMH17 opened this issue 1 year ago • 1 comments

⚠️ This issue respects the following points: ⚠️

Bug description

After the recent upgrade to Nextcloud 28 the theme color for icons isn't applied correctly anymore.

In this picture you can see the folder icon color correctly matches the theme color, archive and document icons ignore the selected color. sample_bugx

Steps to reproduce

  1. Select a custom theme color
  2. Upload a compressed archive
  3. Check the icon color in nextcloud web GUI

Expected behavior

Theme color should be applied consistently to choice.

Installation method

Community Manual installation with Archive

Nextcloud Server version

28

Operating system

Debian/Ubuntu

PHP engine version

PHP 8.2

Web server

Apache (supported)

Database engine version

MariaDB

Is this bug present after an update or on a fresh install?

Update

SMH17 avatar Dec 29 '23 01:12 SMH17

After the update to 28.0.2 the theme color is totally ignored even for folders and selection highlight.

This bug can be reproduced when it is selected "Dark Theme" in "Appearance and accessibility settings".

SMH17 avatar Feb 01 '24 19:02 SMH17

I think this might happen due to some places that still use the primary color instead of the every-where-else-used primary-element color.

susnux avatar Mar 08 '24 18:03 susnux

https://github.com/nextcloud/server/blob/672923f0a6ab5a692273326250fe1394b4e41bd9/lib/private/Preview/MimeIconProvider.php#L85-L88

I guess some icons are still not fixed :see_no_evil: I think we can either:

  1. move mime icons to the default grey or their brand color (like docx being blue, pdf being red...)
  2. add back mime icon theming (I think this doesn't really add much anyway and is over engineered)

skjnldsv avatar May 23 '24 12:05 skjnldsv

In this picture you can see the folder icon color correctly matches the theme color, archive and document icons ignore the selected color.

I have to correct myself. This is by design. Folders are color primary, while documents are by design always that blue. Like pdf is always red, documents are always blue.

susnux avatar May 23 '24 13:05 susnux