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

[Bug][a11y]: Disabled button is showing incorrect font color in High Contrast Mode for some variants #3244

Closed
1 task done
dineshy87 opened this issue May 29, 2023 · 1 comment
Labels
a11y Issues related to accessibility bug Something isn't working Discuss w/ design Needs insight from Spectrum or another designer Spectrum CSS Bug triage An issue needing triage WHCM

Comments

@dineshy87
Copy link

Code of conduct

  • I agree to follow this project's code of conduct.

Impacted component(s)

Button (primary, negative, secondary, quiet)

Expected behavior

All quiet, primary, secondary, negative buttons must use GrayText as font color when they're disabled.

Actual behavior

Some variants like primary, secondary, negative and quiet buttons are showing in ButtonText color instead of GrayText when they're disabled.

Screenshots

Disabled buttons of some variants showing wrong color in hcm

What browsers are you seeing the problem in?

No response

How can we reproduce this issue?

  1. Go to 'https://studio.webcomponents.dev/preview/sNQ3ZM8hLebINbXzl9Vt/src?p=stories&busid=7f405180-fd86-11ed-ab77-e1c7fb078b46'
  2. Open the Chrome browser and navigate to the website you want to test.
  3. Open the Chrome DevTools by pressing Ctrl+Shift+I (Windows) or Cmd+Option+I (Mac).
  4. Click on the "Rendering" tab.
  5. In the Rendering tab, scroll down and find the "Emulate CSS media feature forced-colors" option.
  6. Select the "active" option from the drop-down menu.

The website should now be displayed in forced-colors mode.

Sample code that illustrates the problem

No response

Logs taken while reproducing problem

No response

@dineshy87 dineshy87 added bug Something isn't working triage An issue needing triage labels May 29, 2023
@Westbrook Westbrook added the a11y Issues related to accessibility label Jun 2, 2023
@najikahalsema najikahalsema added Discuss w/ design Needs insight from Spectrum or another designer Spectrum CSS Bug Spectrum CSS WHCM and removed Spectrum CSS labels Jun 2, 2023
@Westbrook
Copy link
Contributor

This delivery has been corrected by updated CSS dependencies along the way, as seen in the OP demo:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Issues related to accessibility bug Something isn't working Discuss w/ design Needs insight from Spectrum or another designer Spectrum CSS Bug triage An issue needing triage WHCM
Projects
None yet
Development

No branches or pull requests

3 participants