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

Extremely difficult to read "disabled"/"grayed-out" text and embedded hyperlinks due to low contrast #112

Closed
CAM-Gerlach opened this issue Oct 13, 2018 · 9 comments
Assignees
Milestone

Comments

@CAM-Gerlach
Copy link

Describe Your Environment

  • QDarkStyle: 2.5.4

  • OS: Windows 8.1 x64

  • Python: 3.6.6 x64

  • PyQt: 5.9.2

  • Qt: 5.9.6

  • QtPy: 1.4.2

  • QT_API: ?

Language

Python (Anaconda)

Description / Steps to Reproduce

We are in the process of implementing a dark theme using QDarkStyle in the Spyder IDE as tracked by spyder-ide/spyder#8068 , and in the course of reviewing the initial implement in @dalthviz 's spyder-ide/spyder#8020 , I noticed a number of issues that Spyder maintainer @ccordoba12 identified as being within the scope of this upstream rather than our specific usage, among them this one, and requested I open this here.

  1. Make some text disabled (i.e. a disabled menu option, checkbox, etc) or a hyperlink
  2. Try (and fail) to read it

Actual Result

Disabled text completely illegible, even by a user with typical vision and a properly calibrated Pro-IPS monitor.

image

Hyperlinks are also almost as hard to read on a dark background:

image

Expected Results / Proposed Result

Disabled text should be lighter, to improve contrast and thus legibility, so users can clearly see what they are missing. Similarly, hyperlinks should use one of the lighter blues in your palette to make them more readable.

Relevant Code

See spyder-ide/spyder#8020

@CAM-Gerlach CAM-Gerlach changed the title Improve contrast for "disabled"/"grayed-out" text and embedded hyperlinks Extremely difficult to read "disabled"/"grayed-out" text and embedded hyperlinks Oct 13, 2018
@CAM-Gerlach CAM-Gerlach changed the title Extremely difficult to read "disabled"/"grayed-out" text and embedded hyperlinks Extremely difficult to read "disabled"/"grayed-out" text and embedded hyperlinks due to low contrast Oct 13, 2018
@dpizetta
Copy link
Collaborator

Hi, I will take care of this. I am preparing a more simple and better palette. I hope this week I would finish!

@dpizetta dpizetta self-assigned this Oct 23, 2018
@CAM-Gerlach
Copy link
Author

@dpizetta Fantastic, thanks so much!

@dpizetta
Copy link
Collaborator

I think I cannot fix the hyperlinks. As they use RichText it seems that the application CSS does not have access to it. If you check the text created with rich text it already has "a {color:blue;}; inside.

Some references below. I think you can choose a median color that works for both, white and black themes I suggested this one: #4D545B, I am using now for text disabled - see pictures below. That original blue is also ugly for white theme :).

screenshot linux 2018-10-25 00 43 58
screenshot linux 2018-10-25 00 49 03
screenshot linux 2018-10-25 00 49 12

The issue about the disabled text was corrected, I think. :)

screenshot linux 2018-10-25 00 52 55

I will upload soon, making some more tests here.

@flutefreak7
Copy link

I was doing workarounds to deal with the disabled text, so I'm very grateful for this fix as well!

@CAM-Gerlach
Copy link
Author

Thanks @dpizetta , sounds good! @dalthviz take note.

@dpizetta dpizetta added this to the 2.6 milestone Oct 25, 2018
@dpizetta
Copy link
Collaborator

Closing for now :)

@CAM-Gerlach
Copy link
Author

Thanks @dpizetta !

@ccordoba12
Copy link
Collaborator

@dpizetta, when do you plan to release a new version of QDarkStyle? I say it because we'd like to use it along with our next Spyder 4 beta.

@dpizetta
Copy link
Collaborator

Hi @ccordoba12 , @CAM-Gerlach , the new release is planned to be tomorrow. This new style has improved some spacing in Spyder also. I think the most annoying issue is that the buttons on tabs - superior part, they use tool buttons, that seems strange. I will try to improve the tool button in the next release, I've already tried by it is not so easy as I thought. There is a new icon package for the dark theme in v4?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants