-
Notifications
You must be signed in to change notification settings - Fork 5
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
label member #39
Comments
Thanks for this @johnfoliot!
Can do. I’m still a little fuzzy on all of the conformance language.
Agreed. Would it help make this clearer in the example if I specifically highlight that in the context of the Manifest, the
I can clarify this. As I mentioned above, the explicit
This is a good question. It likely depends on context. In terms of the Manifest usage, |
No worries, that's why I do reviews :-)
Yes (please).
Cool, thanks.
Hmmm... beyond my skills level as well. However it is critical that any graphical artifact has some type of accessible name for screen reader usage. Failing to provide that will make this inaccessible, but I am unsure of specific implementation guidance (sorry). From my vantage point, all I can do is focus on the criticality. |
@johnfoliot Take a look at #40 and let me know how that language looks. For simplicity, I broke out the accessible name calculation to its own algorithm. Please not that for the worst-case scenario of no accessible name being defined, I have set up the algorithm to return an empty string as the accessible name which—if OSes act similarly to the web at least—should preclude the image from being announced by screen readers in a useless/distracting way. |
@johnfoliot ::gentle nudge:: If this looks good, I’ll get it merged today. |
Sorry Aaron,
WFM - thanks for seeing this through.
JF
…On Mon, Mar 29, 2021 at 2:45 PM Aaron Gustafson ***@***.***> wrote:
@johnfoliot <https://github.com/johnfoliot> ::gentle nudge::
If this looks good, I’ll get it merged today.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#39 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJL44ZVCU5CWDVKMCTM2L3TGDDDTANCNFSM4ZTDCNXA>
.
--
*John Foliot* | Principal Accessibility Specialist
"I made this so long because I did not have time to make it shorter." -
Pascal "links go places, buttons do things"
|
Clarified some of the language as well. Closes #39
APA just confirming ok with this, just shifting labels around accordingly. |
@michael-n-cooper Any idea why the w3c bot reverted your change? |
no :( |
The current draft of the specification states that providing an Accessible Name via the label member is a SHOULD requirement (while allowing for an embedded accessible name).
The text was updated successfully, but these errors were encountered: