-
Notifications
You must be signed in to change notification settings - Fork 2
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
Toolbar colors in light mode in Spyder 5 #42
Comments
Ok, realize that the button background is indicating state, so perhaps the underlying problem is the extent of the difference in background gray level between toolbar/on state and the pane/off state. I'm afraid it just looks ugly to my eyes in light mode! A less dark gray for the toolbar would solve both problems. |
I agree that the light green and blue icons are a little bit hard to see in the light mode. I guess my screen is making things worse since it has a matte display. |
I'm going to move this one to our UX improvements repo because that's where it needs to belong. |
This is a very subjective matter and I doubt there's something we can do about it because it affects a lot of other elements in the interface.
This could be improved, I think. |
Thoughts @isabela-pf? |
For me, it is mostly the green triangle (Run File) that is really hard to see. The icons that are a mix of dark grey and green are not too bad (Run Current Cell, etc...). The contrast for the blue icons (Debug File, Run Current Line, etc...) could be improved I think, but it is not to bad. Overall, I like the design of the new icons a lot. |
So design and colour schemes are subjective, but contrast actually isn't - there are well established ways of measuring contrast ratios and standards that can be applied. w3c sets a standardin the form of WCAG 2.1 for accessibility in web pages. Broadly the recommendation is that text elements have a contrast ratio of 4.5:1 between foreground and background and graphical elements (icons etc) that are used for navigation have 3:1. These levels correspond to comfortable levels for people with 40:20 and 20:20 vision respectively - so we're not talking about very blind people here! Using the screenshots above, GIMP and the first contrast checker I found on google (https://webaim.org/resources/contrastchecker/) I got: The grey background/blue icons in Spyder 5 have a contrast ratio of 1.9:1 - so well below the recommended 3:1. The grey-green combo is worse at 1.33:1. The Spyder 4 combo of lighter grey and darker blue achieves a contrast ratio of 4.3:1 in comparison, which is above the recommended 3:1 and the green on grey combo there is at 4.5:1 In Spyder 5 dark mode the contrast between background grey and blue is 2.57:1 and with the Spyder 2 Iconset the highlighting brings that to 4:1 (depending a bit where you pick the blue). So in summary - for comfortable accessibility it goes: Spyder 5 light mode (fails to meet 3:1) |
I can’t thank you enough for this specific feedback! Trying to keep up with the colors as we kept iterating was messy, so I was sure there were mistakes I’d need to address. While I agree that having a lighter background for the main toolbar in light mode would solve a lot of problems, we had to go with something roughly this value to account for the different types of toolbars and their button states. It will take some reshuffling of colors or work on an alternate pressed state to have a lighter toolbar. I also think it’s worth reworking the blue and greens for icons regardless of how we solve the other contrast issues. It’s now on my list to check contrast with these final colors, so thanks again! |
Thanks for taking this on board so positively - I just wish my University web design team were prepared to listen.... !
Well with my slightly dodgy eyesight contrast checking comes enabled by default in the wet-ware... |
I’ve started working on ideas for changes in the toolbar, most specifically around the icons. This is an area that (as has been kindly pointed out) has several issues and can easily inhibit users since so many major functions live in the main toolbar. At the moment, my goal is to adjust icon colors with the existing background and state colors on the toolbars. The toolbar colors are used throughout the rest of the interface, so changing them will have impact in several other areas as well. I’m think adjusting icons will make them much more usable while scoping the problem so we can resolve it more quickly. (To be clear, I’m determining pass/fail based on the non-text contrast ratio of 3:1. I’ve also added a file icon in every normal icon color before the colored ones to provide an easy comparison.) Active icons on main toolbar backgrounds:Pass in both light and dark mode as-is! No change needed. Disabled icons on main toolbar backgrounds:Fail. The colors are the part I’m most struggling to work out. I thought it was still worth sharing since some parts are working. Green in dark modeCurrent color is Green b70.
Changing it to Green b80 makes it so that only the pressed state fails. Green b90 passes all, so that would be ideal. Blue in dark modeCurrent color is Blue b70.
Changing it to Blue b80 only passes the normal state. Blue b90 only fails the pressed state. Any higher and the icons fail to look blue. Green in light modeThis is where I start to have the most problems. The light mode’s grey toolbar is middle range enough that I am not getting sufficient contrast even with the darker colors. Current color is Green b70.
The lightest green that passes for the normal state with this background is Green b30. Even so, the other two states fail. Blue in light modeCurrent color is Blue b70.
The lightest blue to pass for the normal state is Blue b50, but this fails the other two states. Blue b40 fails only the pressed state. Blue b30 passes all, but is almost indistinguishable from the grey in my opinion. With this, I think it might be best to adjust the background greys used in the light toolbar. I’m worried about what other areas of the UI it might affect since this color is shared, but I haven’t found sufficient contrast for any of these other options in light mode. |
Another round of review determined green in light mode will be green b30 until we adjust the light toolbar to be less dark. Hooray! I'll get this in PR form and link it here. |
This was partially addressed in ColinDuquesnoy/QDarkStyleSheet#335. |
Problem Description
There are some issues with toolbar colours when using Spyder 5.0.x in light mode:
a) The toolbar background is inconsistent between panes and the main toolbar and sometimes within a pane (e.g. find pane):
b) The contrast on the buttons is much worse than in Spyder 4 (and I can't see how to change it) - this can be an accessibility issue as well as just being a little irritating... :-) cf. Spyder 4 below:
Dark mode is better for point (a) but still has issues for point (b). 5.0.1 overall much better than 5.0.0 through) :-)
What steps reproduce the problem?
What is the expected output? What do you see instead?
I would have expected a consistent background colour to the toolbars in the panes and the main toolbar (and certainly not have buttons with a different background to the toolbar!). I would like to have a better contrast, or someway to adjust the contrast.
Versions
Dependencies
The text was updated successfully, but these errors were encountered: