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

opacity issues on disabled controls #629

Closed
Malvoz opened this issue Dec 7, 2021 · 0 comments · Fixed by #630
Closed

opacity issues on disabled controls #629

Malvoz opened this issue Dec 7, 2021 · 0 comments · Fixed by #630
Assignees
Labels

Comments

@Malvoz
Copy link
Member

Malvoz commented Dec 7, 2021

UAs use opacity on disabled controls, but it doesn't work well on the map controls for the following reasons:

  1. Unlike UAs we use aria-disabled and not disabled, so the controls remain focusable, causing a barely visible outline on focus (because the outline too has the opacity applied).

  2. Too low contrast on the Reload button (because it has both a lighter color + opacity as of Refactoring HTML anatomy of items in layers #531).

  3. Creates a stacking context (I believe) on disabled controls in the layer control, causing underlying layer setting buttons to show through the overlaying layer item on drag.

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