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

Icon contrast too low #43

Closed
ovanbiervliet opened this issue Feb 20, 2021 · 7 comments
Closed

Icon contrast too low #43

ovanbiervliet opened this issue Feb 20, 2021 · 7 comments
Labels
bug Something isn't working

Comments

@ovanbiervliet
Copy link

ovanbiervliet commented Feb 20, 2021

The contrast on the icons (settings, background color) is too low for accessibility reasons.

Core-2021-0220 121822@2x

Also happens with lighter background color:
Core-2021-0220 122131@2x

Edit: I saw this behaviour after first time usage. Now I get a dark cog for the settings, although the icon for background color remains nearly invisible.

@ovanbiervliet ovanbiervliet added the bug Something isn't working label Feb 20, 2021
@superhighfives
Copy link
Owner

Thanks for flagging, @ovanbiervliet. Just to clarify, which version of macOS is this on? Some of these are known bugs / quirks in SwiftUI, but the background colour menu icon is a new one. Also, do you have reduce transparency enabled in your accessibility settings?

@superhighfives
Copy link
Owner

Oh, also @ovanbiervliet, did you change between dark and light mode at all? That can also cause some quirks.

@ovanbiervliet
Copy link
Author

Big Sur 11.2.1, no accessibility settings enabled.

Didn't change dark/light after installing. The cog fixed itself after I toggled the Hide Pika setting in the menu.

@ovanbiervliet
Copy link
Author

Just did a quick test: from light to dark and back resulted in the cog being wrong again.

@superhighfives
Copy link
Owner

Yeah @ovanbiervliet, it's a bug in SwiftUI, essentially. Unfortunately I have no control over the display of the icons or their colours, which is why I've used shadows. This seems to only occur in Big Sur. From my experiments, I found:

  • The cog / settings icon has a low opacity on launch. Once you interact with it, it changes to the appropriate colour. I've added some code to toggle it on and off quickly on appear, but it's a hack. Changing from light to dark seems to make the icon permanently low opacity, and it doesn't always seem to work as expected. It's a frustrating one.
  • You can't change the colour of the menu / (...) icons in the swatches, which is why they have a shadow, to try and make them more visible. Nothing I have tried has made the icon colour change, except for swapping between dark and light mode, so it seems to be baked in. The other option would be to place it inside a circle / some kind of background, similar to how the Swap button currently works. That's likely the best solution, but impacts the swatch design.

I'm still unsure how the background icon has a lower opacity there, as I can't recreate it. If you restart the app, is it still lower opacity?

Ultimately I'm keen to see if newer versions of Big Sur resolve these issues, but until then, short of redesigning these components (which is on the roadmap, and I have an issue for that: #19) there's not a lot I can do.

@ovanbiervliet
Copy link
Author

After restarting, the first time I see the Pika window the icons look fine. But as soon as I interact, even trying to take a window screenshot with CleanShot, makes the icons go wrong. Interacting with the individual icons afterwards restores the correct colors again.

@superhighfives
Copy link
Owner

Understood. Keen to check out how it is in the newer versions of Big Sur, and I'm flagging it as part of #28 and #19. 👍 Going to close this in the meantime.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants