Skip to content

Conversation

estelle
Copy link
Member

@estelle estelle commented Sep 1, 2025

Added a dashed outline to the checked state of the question box so that color is not the only change (currently red/green change is not visible to 10% of the population)

Added a dashed outline to the checked state of the question box so that color is not the only change (currently red/green change is not visible to 10% of the population)
@estelle estelle requested a review from a team as a code owner September 1, 2025 10:00
@estelle estelle requested review from chrisdavidmills and removed request for a team September 1, 2025 10:00
@github-actions github-actions bot added Content:CSS Cascading Style Sheets docs size/xs [PR only] 0-5 LoC changed labels Sep 1, 2025
Copy link
Contributor

github-actions bot commented Sep 1, 2025

Preview URLs

Flaws (1)

URL: /en-US/docs/Web/CSS/:state
Title: :state()
Flaw count: 1

  • macros:
    • Macro produces link /en-US/docs/Learn_web_development/Core/Styling_basics/Cool-looking_box which is a redirect

Copy link
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but you should also make the same change in the same example code found at https://developer.mozilla.org/en-US/docs/Web/API/CustomStateSet#matching_a_custom_state_in_a_shadow_part_of_a_custom_element, which is referenced from the page you have edited.

@estelle
Copy link
Member Author

estelle commented Sep 3, 2025

see #40995

Copy link
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect, thanks. This is now GTG.

@chrisdavidmills chrisdavidmills merged commit 52463ee into main Sep 4, 2025
11 checks passed
@chrisdavidmills chrisdavidmills deleted the estelle-patch-6 branch September 4, 2025 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:CSS Cascading Style Sheets docs size/xs [PR only] 0-5 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants