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

Updated the characteristics table of "caption" role #1975

Merged

Conversation

giacomo-petri
Copy link
Contributor

@giacomo-petri giacomo-petri commented Jul 7, 2023

Proposal:

while reviewing the specs for the caption role, I've realised that the characteristics table, in particular the "Required Accessibility Parent Roles" row, is not fully reflecting the requirements listed above.

  • I've updated the "Required Accessibility Parent Roles" table content, adding also "group" and "radiogroup".
  • I've also sorted the list of "Required Accessibility Parent Roles" by alphabetical order (both in text and in table).

Closes: #1974


Diff

…s above

Updated the Characteristics table for the caption role, reflecting the specs previously declared.

Also sorted the list of "Required Accessibility Parent Roles" by alphabetical order.
@pkra pkra added this to the ARIA 1.3 milestone Jul 13, 2023
@spectranaut spectranaut self-requested a review July 13, 2023 17:22
@pkra pkra added the editorial a change to an example, note, spelling, grammar, or is related to publishing or the repo label Jul 13, 2023
spectranaut
Copy link
Contributor

@spectranaut spectranaut left a comment

Choose a reason for hiding this comment

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

Thanks for this!

@spectranaut spectranaut merged commit 411bd50 into w3c:main Aug 7, 2023
github-actions bot added a commit that referenced this pull request Aug 7, 2023
SHA: 411bd50
Reason: push, by spectranaut

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
jnurthen pushed a commit that referenced this pull request Oct 10, 2023
while reviewing the specs for the caption role, I've realised that the characteristics table, in particular the "Required Accessibility Parent Roles" row, is not fully reflecting the requirements listed above.

    I've updated the "Required Accessibility Parent Roles" table content, adding also "group" and "radiogroup".
    I've also sorted the list of "Required Accessibility Parent Roles" by alphabetical order (both in text and in table).

Closes: #1974
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
Projects
None yet
4 participants

Follow Lee on X/Twitter - Father, Husband, Serial builder creating AI, crypto, games & web tools. We are friends :) AI Will Come To Life!

Check out: eBank.nz (Art Generator) | Netwrck.com (AI Tools) | Text-Generator.io (AI API) | BitBank.nz (Crypto AI) | ReadingTime (Kids Reading) | RewordGame | BigMultiplayerChess | WebFiddle | How.nz | Helix AI Assistant