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/clarify understanding for 1.4.4 Resize Text, harmonise 1.4.8 Visual Presentation #2270

Merged
merged 13 commits into from
Aug 7, 2024

Conversation

patrickhlauke
Copy link
Member

Relates to #2169 and questions about whether a site that break using "text-only" resizing fails, even if zooming works

Relates to #2169 and questions about whether a site that break using "text-only" resizing fails, even if zooming works
@detlevhfischer
Copy link
Contributor

detlevhfischer commented Mar 21, 2022

Rendered view via githack [Updated]

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.

How about breaking the exception into its own sentence? For example:

The intent of this Success Criterion is to ensure that visually rendered text, including controls and labels using text, can scale larger so that they can be read more easily by people with milder visual impairments, without requiring the use of assistive technology (such as a screen magnifier). There is an exception for images of text and captioning, since those technologies might not provide for machine readable text. Users may benefit from scaling all content on the Web page, but text is most critical.

@patrickhlauke
Copy link
Member Author

@bruce-usab i like your rewrite there. so is that exception there (about images of text and captions) what the cryptic "in data form such as ASCII" actually means to mere mortals?

removes the inscrutable bit about "ASCII", and mentions the exception (which was otherwise missing from the understanding)
@patrickhlauke
Copy link
Member Author

took @bruce-usab's suggestion (with a slight bit of language tweaking), which now nicely also mentions the exception (where previously the document is mostly silent about it, except for a bit about images of text).

this also removes the cryptic mention of "text characters that are still in data form such as ASCII" which i still fail to understand. again, if anybody wiser than me has any idea what that meant, and why it was there/if it's essential to bring that back, happy to reinstate something, but it'll need a bit of clarification as well, as currently it seems to baffle people.

@JAWS-test
Copy link

@patrickhlauke maybe you also want to remove the ASCII part of 1.4.8? - it is the same there

@patrickhlauke
Copy link
Member Author

@JAWS-test good point, might as well

@patrickhlauke patrickhlauke changed the title Expand/clarify understanding for 1.4.4 Resize Text Expand/clarify understanding for 1.4.4 Resize Text, harmonise 1.4.8 Visual Presentation Mar 22, 2022
@bruce-usab
Copy link
Contributor

I do not know who wrote text characters that are still in data form such as ASCII but I am guessing it is a reference to situations where ASCII text is being used (say, a timed-text file for captioning) but there is no feasible way (due to constraints of the video player) to provide the end-user a means to control the font size. This is less of a problem nowadays. I agree that it is not helpful to mention ASCII per se.

@patrickhlauke
Copy link
Member Author

@alastc any chance this PR and #2194 could be reviewed/discussed/merged soon?

@yatil
Copy link
Contributor

yatil commented Jul 6, 2022

Could we somehow form a community group that reviews and merges those changes largely independently from AG WG? It feels like AG WG has not enough time to prioritize this, and a dedicated group for these editorial changes would probably be more efficient, especially if the red tape is removed.

@patrickhlauke
Copy link
Member Author

I was under the impression there was already a sub-group (meeting on Fridays) of the AGWG supposed to look at clearing the backlog of some of the pre-2.2 related issues, but I think that's been going slowly as well. agree that for reasonably non-controversial additions/edits, it would be great to have a more nimble approach...

@patrickhlauke
Copy link
Member Author

@alastc @mbgower ping?

Copy link

netlify bot commented Jul 19, 2024

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit 8b746eb
🔍 Latest deploy log https://app.netlify.com/sites/wcag2/deploys/66b3ee97afa6a20008104500
😎 Deploy Preview https://deploy-preview-2270--wcag2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mbgower mbgower merged commit 67ef3f3 into main Aug 7, 2024
5 checks passed
@mbgower mbgower deleted the patrickhlauke-understanding-resize-text branch August 7, 2024 22:16
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.

8 participants