Skip to content
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 Correct Foreground Colors for Texts in Dark and Light Modes #46698

Closed
alperozturk96 opened this issue Jul 23, 2024 · 7 comments
Closed

Use Correct Foreground Colors for Texts in Dark and Light Modes #46698

alperozturk96 opened this issue Jul 23, 2024 · 7 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap design Design, UI, UX, etc. enhancement feature: files needs info wontfix This will not be worked on

Comments

@alperozturk96
Copy link

alperozturk96 commented Jul 23, 2024

Problem

Recently modified files are highlighted well in dark mode, but files modified some time ago use a grey color that doesn't provide good contrast.

Screenshot 2024-07-23 at 09 44 22
Screenshot 2024-07-23 at 09 44 40

Solution

Corresponding foreground color must be used for icons and texts.

@alperozturk96 alperozturk96 added enhancement 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jul 23, 2024
@alperozturk96 alperozturk96 changed the title Use Correct Foreground Colors for Text and Icons in Dark and Light Modes Use Correct Foreground Colors for Texts in Dark and Light Modes Jul 23, 2024
@tobiasKaminsky
Copy link
Member

Android: nextcloud/android#13309
Android notes: nextcloud/notes-android#2300

@susnux
Copy link
Contributor

susnux commented Jul 24, 2024

This is done on purpose:
Older files get slightly less intense colors to reflect their age.
This color is based on the modification time and range from maxcontrast to main text.

CC @nextcloud/designers if we should remove this feature.

@susnux susnux added design Design, UI, UX, etc. needs info labels Jul 24, 2024
@susnux
Copy link
Contributor

susnux commented Jul 26, 2024

Android: nextcloud/android#13309
Android notes: nextcloud/notes-android#2300

No this are different issues, this one is about a special files feature to adjust the color based on how old the file is.

In general we have two forground colors main-text and maxcontrast, in the past we used to make icons maxconstrast but from design reviews we changed icons to main text color, so probably for the linked issue just also make the icon main text color?

@nimishavijay
Copy link
Member

This is done on purpose: Older files get slightly less intense colors to reflect their age. This color is based on the modification time and range from maxcontrast to main text.

I would vote for removing this in the interest of maintaining consistency of colors.

We can start off with removing the range of colors and sticking to --color-main-text for recent files and --color-text-maxcontrast for older files as the result of the color mix is often too subtle to notice anyway.

@skjnldsv
Copy link
Member

I would vote for removing this in the interest of maintaining consistency of colors.

We do have the same for file sizes. The larger the file, the stronger the size color will be. This is an old feature that many uses. ⚠️

@marcoambrosini
Copy link
Member

marcoambrosini commented Jul 29, 2024

I think that everything should be of the same color, same as the file name. I don't think the feature is helpful because it's not easy to understand what these color difference indicate.

@jancborchardt
Copy link
Member

We do have the same for file sizes. The larger the file, the stronger the size color will be. This is an old feature that many uses. ⚠️

Exactly, as @susnux and @skjnldsv said this was intentionally added a long time ago and is a nice differentiator for us compared to other solutions as well.

  • It helps distinguish newer (aka more relevant) files even if the list is sorted alphabetically
  • It also directly highlights larger (using more space if you need to clean up for example)

So no, we should not remove this. (FYI @nimishavijay @marcoambrosini) And as said the contrast is fine since the lowest we go is color-text-maxcontrast.

We also do a similar thing with the file extensions in the file list, because they are not as important as the file name itself. It helps to focus.

@jancborchardt jancborchardt added the wontfix This will not be worked on label Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap design Design, UI, UX, etc. enhancement feature: files needs info wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

8 participants