-
Notifications
You must be signed in to change notification settings - Fork 38
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
Pencil button suffers from low contrast #338
Comments
The problem is that it is a disabled button and should not be that bold. Another problem might be that this button is invisible if the custom model is not enabled and so this is a bit hidden. But this makes sense from a functional perspective. Not sure how to improve it because clicking on it and enabling the custom model might be counter-intuitive (?)
I'm unsure why the icons you refer are better in this regard. The current icon could be also interpreted as a pencil next to a rectangle. |
I would say that other elements also suffer from this problem, but to a lesser extent.
Maybe you could find a different mechanic to mark enabled elements instead of using grey for disabled ones. You're already using a blue shade to highlight the active profile. Maybe you could alternate between black and a blue? Changing the background color might also be a valid approach to increase contrast. Note that I have good eyesight, so this is a minor inconvenience for me. However, visually impaired users may not be able to fully use the interface.
I wouldn't change that to be honest.
The current icon is the widely used for text editors and text input boxes: That's why i would rather use something like this instead: https://fontawesome.com/icons/draw-polygon?f=classic&s=regular |
I only discovered this button because I stumbled across your blog post.
Using light grey on a white background isn't ideal. The icon itself isn't much help either. It's a pencil next to a text box, which doesn't really convey the idea of "click here to draw polygons".
I'd suggest using more distinctive icons like geojson.io:
The text was updated successfully, but these errors were encountered: