Updates to flashing#628
Conversation
✅ Deploy Preview for wcag3 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
@kfranqueiro I noticed that with a rename (of the shortname), it breaks the links to the github issues, where do I fix that? |
(Edit: this has been resolved) It looks like you already renamed the labels in GitHub, which will break the links in the existing Working Draft. Instead, revert the labels to their original names in GitHub, and instead set References:
(Edit: this has been resolved) Meanwhile, I would point out that despite "over threshold" being added to both titles, the "no exceptions" requirement text still says "Content does not include flashing", with no reference to the threshold. (Even within the definition of flashing, the threshold seems to only be brought up within a note, i.e. non-normative text.) |
| issueLabel: No flashing (no exceptions) | ||
| --- | ||
|
|
||
| All :term[flashes] are below the :term[general flash and red flash thresholds] without a minimum size. |
There was a problem hiding this comment.
Not sure the latter part of this is clear. What is 'without a minimum size'?
| ::: | ||
|
|
||
| :::note | ||
| If there is an :term[accessibility supported] method of setting a user-preference to prevent flashing, the content can be considered to avoid flashing if that preference is respected. |
There was a problem hiding this comment.
Is this a method or a note?
| type: foundational | ||
| --- | ||
|
|
||
| Content does not include a change of contrast (over 4.5:1) going from light to dark (or dark to light) of more than 30% of the visible content area (viewport). |
There was a problem hiding this comment.
| Content does not include a change of contrast (over 4.5:1) going from light to dark (or dark to light) of more than 30% of the visible content area (viewport). | |
| Content does not include a single :term[flash] of more than 30% of the visible content area. |
The original includes some ambiguity:
- How quick a change of contrast?
- Contrast between what and what?
Also, note sure 'viewport' is needed. If it is then it would just be '30% of the viewport' if that isn't clear enough then the existing text (without viewport) would be ok.
Finally, having looked at the definition for 'flash' I think it might need some attention in light of this but also, I suspect, from the no-flashing provisions.
There was a problem hiding this comment.
And I note the 'ednote' now!
3 participants
From the comments on the document, and the survey.