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

Update three-flashes-or-below-threshold.html #2127

Merged
merged 3 commits into from
Jan 4, 2022
Merged

Conversation

mbgower
Copy link
Contributor

@mbgower mbgower commented Nov 8, 2021

A first attempt at rewriting flashing assessment criteria

x-ref #1132

Closes #585

A first attempt at rewriting flashing assessment criteria
@patrickhlauke
Copy link
Member

while you're in that general area, might also be worth updating the links/resources that have relocated.

<a href="http://trace.wisc.edu/peat/">Trace Center Photosensitive Epilepsy Analysis Tool (PEAT)</a>

change to

<a href="https://trace.umd.edu/peat/">Trace Center Photosensitive Epilepsy Analysis Tool (PEAT)</a>

and likewise

<a href="http://trace.wisc.edu/peat/photosensitive.php">Information about Photosensitive Seizure Disorders</a>

change to

<a href="https://trace.umd.edu/information-about-photosensitive-seizure-disorders/">Information about Photosensitive Seizure Disorders</a>

I'd note that PEAT as a tool is also fairly useless nowadays in a world of responsive web design, as it effectively can only analyse content at a resolution of 1024x768 and does so by resizing your browser (where it can) to that size. might be worth noting somehow?

@patrickhlauke
Copy link
Member

ah, i see i'm flogging my own dead horse with my comment about PEAT ... i may have mentioned it once or twice before, sorry.

also x-ref #553 and #585

provided changes to address patrick's comments.
@patrickhlauke
Copy link
Member

i've done a separate PR for updating the links (as mentioned in #2127 (comment)) #2132 (as that's a more straightforward change that is, likely, non-controversial)

@alastc
Copy link
Contributor

alastc commented Nov 30, 2021

i've done a separate PR for updating the links (as mentioned in #2127 (comment)) #2132 (as that's a more straightforward change that is, likely, non-controversial)

Your PR for that was merged in, so no need to replicate here.

evaluation. The 341 x 256 pixel block represents a 10 degree viewport at a typical
viewing distance. (The 10 degree field is taken from the original specifications and
represents the central vision portion of the eye, where people are most susceptible
to photo stimuli.)
</p>
<p>With the proliferation of devices of varying screen sizes (from small hand-helds to large living room displays), as well as the adoption of <a href="https://www.w3.org/TR/WCAG21/#dfn-css-pixels">CSS pixels</a> as a density-independent unit of measurement, the prior assessment criteria seem outdated. However, an image of a consistent size uses up relatively the same percentage of a user's visual field on any device. On a large screen, the image takes up less size, but the large screen takes up a larger part of the visual field. On a mobile screen, the image may take up most or all of the screen; however, the mobile screen itself takes up a smaller portion of the user's visual field. So the same dimension of the flashing content, represented in CSS pixels can still provide a consistent means of assessment. Substituting CSS pixels for the original pixel block means that the assessment becomes 341 x 256 CSS pixels, or a flashing area of 87,296 CSS pixels.</p>
Copy link
Member

Choose a reason for hiding this comment

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

the prior assessment criteria seem outdated reads a bit weird, especially since the next sentence clarifies why they're not. maybe the prior assessment criteria *may* seem outdated

Copy link
Member

Choose a reason for hiding this comment

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

the assessment becomes 341 x 256 CSS pixels reads slightly off. maybe more accurately the threshold for the combined area of flashing becomes ...

@patrickhlauke
Copy link
Member

note that this still leaves questions open like #553 (comment) (about that relative luminance, and that 0.80.

however, i do think this closes #585 for me, so i'll add the github incantation to the top comment here

@@ -38,12 +38,15 @@ <h2>Intent of Three Flashes or Below Threshold</h2>
did not allow any flashing (even of a single pixel) within a broad frequency range
(3 to 50 Hz). This Success Criterion is based on existing specifications in use in
the UK and by others for television broadcast and has been adapted for computer display
viewing. The 1024 x 768 screen is used as the reference screen resolution for the
viewing. In WCAG 2.0, the 1024 x 768 screen was used as the reference screen resolution for the
Copy link
Member

Choose a reason for hiding this comment

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

a 1024 x 768 screen

changes to accommodated feedback from Patrick and AWK
@alastc alastc merged commit 547d2ac into main Jan 4, 2022
@patrickhlauke patrickhlauke deleted the flashing-measurement branch January 4, 2022 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WCAG 2.1 Understanding 2.3.1 - missing/vague dimension definitions
4 participants