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

editorial: update html-aam note #2503

Merged
merged 9 commits into from
Apr 3, 2025
Merged

Conversation

MelSumner
Copy link
Contributor

@MelSumner MelSumner commented Mar 28, 2025

Closes AccName 250

I am making this change in an effort to make it more obvious that readers should be checking HTML-AAM in specific instances. Due to the recent issues filed against SVG-AAM wherein it's made clear that SVG-AAM should be referring to AccName instead, I've removed the note's reference to SVG-AAM.

Before:

See HTML-AAM, SVG-AAM, or other host language documentation for more information on markup that defines a text alternative.

CleanShot 2025-03-28 at 16 23 32@2x

After:

Important! The accessible name for native HTML elements may be calculated differently. Make sure to review HTML-AAM for more information.

CleanShot 2025-03-28 at 16 34 33@2x

Copy link

netlify bot commented Mar 28, 2025

Deploy Preview for wai-aria ready!

Name Link
🔨 Latest commit 9160fcb
🔍 Latest deploy log https://app.netlify.com/sites/wai-aria/deploys/67eaa69cb10fc80008a14dc4
😎 Deploy Preview https://deploy-preview-2503--wai-aria.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.

@MelSumner MelSumner requested a review from rahimabdi March 28, 2025 21:38
@MelSumner MelSumner added the editorial a change to an example, note, spelling, grammar, or is related to publishing or the repo label Mar 28, 2025
Co-authored-by: Rahim Abdi <[email protected]>
@scottaohara
Copy link
Member

approved, but calling out that #2486 and this PR are making updates in the same area. so it might make sense to combine these to avoid conflicts

@pkra pkra changed the title Update html-aam note editorial: update html-aam note Apr 3, 2025
@w3cbot
Copy link

w3cbot commented Apr 3, 2025

jnurthen marked as non substantive for IPR from ash-nazg.

@pkra pkra merged commit 932fc84 into main Apr 3, 2025
6 of 7 checks passed
github-actions bot added a commit to giacomo-petri/aria that referenced this pull request Apr 8, 2025
SHA: 932fc84
Reason: push, by giacomo-petri

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial a change to an example, note, spelling, grammar, or is related to publishing or the repo spec:accname
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[accname] update spec with note and link to html-aam
6 participants