-
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
Spyder 2 icon set incomplete in Spyder 5.0.1 #43
Comments
Hi @gb119, For Spyder 5 we did a refactoring in the icon manager of Spyder and we noticed that in fact the set of icons from Spyder 2 is incomplete, what was happening is that there was a bug in the old code that did not load the red Can you please explain to me why you want to set Spyder 2 icon theme instead of using the default one? We didn't knew any user that was using this option so it will help us to take decisions regarding this topic.
Also, can you please upload screenshots with some examples of the icons that are not working for you? Thanks for reporting! |
Well the trivial answer is because the option is there in the preferences! So my problem is that I struggle with low contrast - I'm not so blind to need the official high contrast colour schemes, but I notice when the contrast gets a bit low. The Spyder 2 icon set has slightly bigger glyphs and they have highlight hints with a lighter colour - both of which makes it slightly less eye-strain inducing to see them. The Spyder 3 sets are just a little lower in contrast and a bit smaller. It's worse in the light mode in Spyder 5 because the main toolbar background grey is that little bit darker than in Spyder 4 which has dropped the contrast a bit. The screenshots here and in [#15326 illustrate the difference in contrast between Spyder 2 iand 3 icon sets and between Spyder 4 and 5. Although Spyder is not a webpage (! :-) ), the WCAG 2.0 guidenlines for contrast levels are, I think, a good thing to aim for. |
Moving to UX improvements because this is related to accessibility. |
Thanks for pointing this out! My limited understanding of why this is happening is a combination of what @steff456 said—that some icons were already missing in that set—and that we also intentionally replaced some of them with new custom icons that were not a part of any icon font since this would take advantage of the existing path and ensure they no longer needed a fallback. (Someone who knows more please correct me if I’m wrong about this.) So I think it should definitely be possible to put back the Spyder 2 icons we intentionally replaced (we’d have to change the way we call the custom icons to make this work), but I would have to check what other icons were missing from the Spyder 2 set in the first case to figure out what state that was in. You make a good point; if we are going to have the option for using those icons we should probably know more clearly what we are offering. |
The option is really a fallback when we're unable to load icons using QtAwesome, which seems to happen still on Windows because it doesn't allow to read fonts not properly installed in the system. Once we're confident that's no longer the case or we find a workaround for it, my plan is to remove the Spyder 2 set because we don't have the time to maintain two icon sets. |
Useful link about this: matplotlib/matplotlib#12954 |
I'm having this issue with Spyder 5.4.2, MacOS 12.4. |
@mpeters2, the Spyder 2 icon set is only a fallback in case we're unable to load the current one, so that at least some icons are displayed on the interface (instead of almost none). So, please don't use it if the Spyder 3 set is working fine for you (it should be on Mac). |
Issue Report Checklist
conda update spyder
(orpip
, if not using Anaconda)jupyter qtconsole
(if console-related)spyder --reset
Problem Description
Some icons are missing when using the Spyder 2 icon set in preferences - in particular I noticed in the find window:
cf. Spyder 3 Icon set:
(5.0.1 is a great improvement on 5.0.0 though btw - great to have it released now!)
What steps reproduce the problem?
What is the expected output? What do you see instead?
Would expect Spyder 2 icon set to be complete or not to be an option.
(I find that Spyder 5 doesn't look as nice in light mode as Spyder 4 did as a result of the switch to Material Design, in dark mode it looks ok, but then I find the contrast in the Spyder 3 icons (darkish blue on black) to be difficult to see (due to a minor visual impairment), Spyder 2 icons are a little better.)
Versions
Dependencies
The text was updated successfully, but these errors were encountered: