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

Expand 1.4.10 Reflow note about 200% and breakpoints, add matching note to 1.4.4 Resize Text #2630

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

patrickhlauke
Copy link
Member

@patrickhlauke patrickhlauke commented Aug 22, 2022

  • Updates the wording in the Reflow Understanding document which discusses the connection between Reflow and Resize Text in attempt to make it clearer
  • Adds similar reciprocal language into the Resize Text Understanding document

Closes #1839

Related: #2101 #704

@patrickhlauke
Copy link
Member Author

Putting this down as an initial strawman change. Happy for any improvements/suggestions to possibly take this further.

@bruce-usab
Copy link
Contributor

bruce-usab commented Aug 22, 2022

Can we change the 2nd 200% in first example sentence to use px?

For example, if at the default browser setting of 100% zoom, text is set at 20px when the window is 1280px wide, the same 20px at 200% zoom would mean a text size of 200%.

(I am holding my breath to see if that [second] 200% becomes 40px or 28.3px.)

Copy link
Contributor

@bruce-usab bruce-usab left a comment

Choose a reason for hiding this comment

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

Second 200% -> 28.3 px

@patrickhlauke
Copy link
Member Author

not sure i follow where you got the 28.3px from.

that whole sentence, which I left mostly untouched, is a bit dense and confusing though. will see if i can make it clearer what's actually going on there.

@patrickhlauke
Copy link
Member Author

Done. reworded to possibly make a bit more explicit what the 20px at 100% / 10px at 400% was getting at.

@JAWS-test
Copy link

I am unsure if this is the intention of the SC. I think the reason for 1.4.10 is,

  • to allow 400% zoom,
  • to simplify reading at 400% zoom by not requiring scrolling.

Low vision users may need a high zoom factor to read content.

Therefore the first reason (400% zoom) is much more important than the second (no scrolling). Unfortunately, 1.4.10 is worded to focus only on the second reason, but the Understanding explains that the intent is actually primarily the first reason.

If we don't require 400% zoom now, we give up the point of 1.4.10. Then SC 1.4.4 and combination with SC 1.4.8 ("Text can be resized without assistive technology up to 200 percent in a way that does not require the user to scroll horizontally to read a line of text", at level AA instead of AAA) would be sufficient.

I am aware that Understanding already says that 200% magnification at 400% zoom is sufficient. In response to my previous inquiries about the meaning of this sentence, I was told here in the forum that this exception only applies to large text (such as headings), which would become much too large at 400%.

@bruce-usab
Copy link
Contributor

bruce-usab commented Aug 23, 2022

not sure i follow where you got the 28.3px from.

That would be 20*sqrt(2) (approximately).

I think at one point there was discussion if 200% meant twice the ~number of pixels or twice the font size. (Since a square 40px on at side has 4x the area as a 20px sided square.) I was not entirely clear where AGWG landed on that detail.

...the same 20px at 200% zoom would mean a text size of 200%.

Should this not be?

...the same 20px at 200% zoom would mean a text size of 40px.

Examples based on 16 css px would be a little more typical.

@patrickhlauke
Copy link
Member Author

not sure i follow where you got the 28.3px from.

That would be 20*sqrt(2) (approximately).

I think at one point there was discussion if 200% meant twice the ~number of pixels or twice the font size. (Since a square 40px on at side has 4x the area as a 20px sided square.) I was not entirely clear where AGWG landed on that detail.

First I heard of it in the context of "text size" (which usually refers to the height/x-height/etc of a font). It may have come about from other discussions about target size and/or size of enhanced focus area (where we do want to talk about area)

...the same 20px at 200% zoom would mean a text size of 200%.

Should this not be?

It's a truism, but one that was there to begin with. And it then sets up the contrast to the next sentence where that isn't the case due to the new media query.

...the same 20px at 200% zoom would mean a text size of 40px.

This whole thing would then veer into disambiguating between CSS pixels and display pixels, and at which display density that would be true. maybe something like "would mean text is displayed at an effective size of 40px (because it's not the actual measure that changes, but the way it's displayed/the CSS pixels are converted to display pixels)

Examples based on 16 css px would be a little more typical.

but make slightly uglier numbers...

@patrickhlauke
Copy link
Member Author

patrickhlauke commented Aug 23, 2022

also, you seem to be commenting on the original text still, rather than my curent PR text? my PR changes that bit about "at 200% zoom it will be 200%" to a slightly less silly sounding

at 200% zoom it will visually appear at twice the size

@bruce-usab
Copy link
Contributor

bruce-usab commented Aug 23, 2022

also, you seem to be commenting on the original text still, rather than my curent PR text?

That must have been the case, my apologies.

...to a slightly less silly sounding
at 200% zoom it will visually appear at twice the size

This version is not silly sounding at all IMHO. Still, my preference would be to say 40px rather than twice the size. It is good enough though, and better than what we had before.

@GreggVan
Copy link

GreggVan commented Aug 23, 2022 via email

@JAWS-test
Copy link

@GreggVan

If I need 400% zoom, then

  • I can't perceive the contents if at 400% zoom contents are not visible or are only enlarged to 200%.
  • I find it very difficult to perceive the content if I have to scroll horizontally after each line.
  • I can perceive the content only slightly harder if I have to scroll horizontally after a block of content.

The difference between NOT PERCEIVING and DIFFICULTLY PERCEIVING seems important to me.

@JAWS-test
Copy link

See #658 (comment) and #1050 (comment)

The exception that content does not need to be zoomed four times at 400% zoom was intended for large text like headlings. Unfortunately, this information never made it into Understanding, although it is completely incomprehensible to me why not, because it's the crucial information for 1.4.10 - otherwise 1.4.10 is worthless in terms of 400% zoom.

@detlevhfischer
Copy link
Contributor

@JAWS-test The 1.4.10 Reflow SC has nothing on differentiating between body text and headings and nothing on requiring 400% body text. (That line would be difficult to draw anyway.) Yes, there is a user need for zooming text to 400%, but it is not a normative requirement (yet). I think it is important to distinguish between "what this SC ideally means" and what is put down in the normative text.
I think @patrickhlauke s explanatory notes are spot on and help clarifying the application of both 1.4..4 and 1.4.10.

@JAWS-test
Copy link

@detlevhfischer Yes, I am aware of that. But I think this is a mistake, because then according to 1.4.4 I can support 200% zoom and at 400% zoom I can theoretically even shrink the font four times to comply with 1.4.10 Reflow. However, this contradicts the intention formulated in the first sentence of the understanding:

The intent of this Success Criterion is to support people with low vision who need to enlarge text and read it in a single column. When the browser zoom is used to scale content to 400% ...

@mbgower
Copy link
Contributor

mbgower commented Apr 5, 2024

I think we should include some illustrations, especially in the resize text Understanding document.

@mbgower
Copy link
Contributor

mbgower commented Apr 5, 2024

@JAWS-test regarding you're comments, I just wanted to ensure you were aware of what is currently stated under "The relation of Reflow to the Success Criterion 1.4.4 Resize Text" in the existing Reflow Understanding.

@bruce-usab
Copy link
Contributor

bruce-usab commented Apr 12, 2024

Briefly mentioned on TF call 4/12. This PR should be part of Reflow discussion.

Even briefer mention on call 4/29.

@w3c w3c deleted a comment from Leeze555 Apr 29, 2024
Copy link
Contributor

@bruce-usab bruce-usab left a comment

Choose a reason for hiding this comment

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

Straight forward improvements. But I would rather PX be the focus for the unit of measure (rather than percentage).

@patrickhlauke
Copy link
Member Author

patrickhlauke commented Apr 29, 2024

@bruce-usab when zooming, the CSS px measures don't change (it's only the browser's internal mapping of "how many actual display pixels do i use to represent a single CSS pixel" changes). so i really don't think it'd be helpful to start talking about px here. (i.e. if i have text sized to 20 px, and zoom to 200% and then query - in the browser itself - what the new dimension of the pixel is, it'll tell me it's still 20 px)

also, refreshing my mind about your "28.3 pixels" thing early on, note that when zooming to 200%, you're not zooming with some square root calculation ... you're doubling the horizontal and vertical dimensions, i don't believe any implementation of zooming considers the actual area of things, but just changes the horizontal and vertical dimensions of how a CSS pixel is represented by that particular factor.

@w3c w3c deleted a comment from Leeze555 Apr 30, 2024
@patrickhlauke
Copy link
Member Author

@Leeze555 can you just ... shush and go away?

@w3c w3c deleted a comment from Leeze555 May 1, 2024
@mbgower mbgower self-assigned this May 3, 2024
@patrickhlauke
Copy link
Member Author

Will see if I can add an illustration/diagram somehow (and clarify how the "at 400% zoom" relates back to the original size, not "320 CSS px and THEN 400%")

Co-authored-by: Scott O'Hara <scottaohara@users.noreply.github.com>
@patrickhlauke
Copy link
Member Author

@scottaohara tweaked and applied your suggestion, and then made the same change to the other file as well to match it.

@detlevhfischer
Copy link
Contributor

Mulling this note, which is a good clarification in substance, I still have some questions as to where it belongs and what it means for UA without windows and UA-based zoom function. The note states:

As with most other Success Criteria, this criterion applies to each variation of the page that is automatically presented for various screen sizes (e.g. media query variations in a responsive site).

I wonder about the "automatically presented" - because it sounds tailored to user agents that have no windows (where the size and/or zoom factor may be changed any time) but use the entire screen as viewport (i.e., phones, tablets). And here we are in a situation where reflow may not work (or not be testable) when the UA does not offer a zoom function triggering reflow the same way as in desktop browsers. So the note does not really clarify if and how this SC would then apply in such windowless environments. Should content reflow when the viewport changes due to other applications loaded alongside the browser split screen mode, or when device orientation changes change the viewport width? Maybe this is another issue that must be addressed elsewhere.

The second thing is that the note talks most about the effect of reflow on text size and what that means related to 1.4.4. That's fine - but is this then not a note that would really belong to 1.4.4, with a note here (in 1.4.10) merely stating that the reflow requirement (as stated normatively) is independent from the text resizing requirement, and refer to the discussion of text size under different viewpoint width /media quer conditions in 1.4.4?

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

Successfully merging this pull request may close these issues.

Understanding for 1.4.10 redefines normative requirement for 1.4.4
7 participants