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.

Sign up for GitHub

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

Jump to bottom

Remaining elements to revise mapping #373

Closed
21 tasks done
scottaohara opened this issue Mar 7, 2022 · 3 comments · Fixed by #426
Closed
21 tasks done

Remaining elements to revise mapping #373

scottaohara opened this issue Mar 7, 2022 · 3 comments · Fixed by #426
Labels
Element Mappings
Milestone
ARIA 1.2 Related

Comments

@scottaohara
Copy link
Member

scottaohara commented Mar 7, 2022 •
edited
Loading

Closing #229, #183 and #346 in favor of this new issue to simplify.

The following elements should map to generic, but we will either need a call out here, or in CORE AAM how AT should treat an 'inline' vs a 'block'/other display type generic (if not doing something with w3c/aria#996).

  • area no href
  • b
  • i
  • data
  • samp
  • kbd
  • pre
  • q
  • s
  • small
  • u
  • var (or maybe this is also role=code? )

the following need to be changed to 'no corresponding role' until their mappings can be finalized - covered in #376

  • dd
  • dt

questions on the following

  • bdi / bdo (change to generic or leave as a 'no corresponding role'?)
  • cite (change to generic or leave as a 'no corresponding role'?)
  • abbr (change to generic or leave as a 'no corresponding role'?)
  • address (change to generic or map to group?)
  • figcaption still map to caption even though it won't provide the figure a name anymore? (i think it might be ok, but want others opinions)
  • label, this stay as no corresponding role? We aren't moving forward with a label role, are we? Could this actually be another instance of caption? is this a generic if not?
  • legend same as above?
@scottaohara scottaohara mentioned this issue Mar 7, 2022
Merged
scottaohara added a commit that referenced this issue Mar 12, 2022
@scottaohara
correct dl/dt roles
75a43f9 
related to resolving #373.  These elements presently have no corresponding role, and their previous mappings weren't correct.  new roles are coming in future aria, see #347
This was referenced Mar 12, 2022
Closed
Closed
Open
@jnurthen
Copy link
Member

Proposed Deep Dive for June 30

@scottaohara
Copy link
Member Author

scottaohara commented Jun 30, 2022 •
edited
Loading

From today's deep dive:

area no href - generic
b - generic
i - generic
kbd - new role (not mapped presently) - create new issue
pre - generic
q - generic
s - map to del? - create issue.
small - generic
u - generic
var - probably code? but create new issue.

bdi/bdo - generic
cite - new issue to discuss if this needs a new role.
address - group

create a new issue to merge caption, legend, and label as they all do the same thing at a high level - identify content that names another element

while <b>, and <i> were resolved to map to generic today, there was discussion about how end users will perceive no different visual representation to those element compared to <strong> and <em> - which are not generic.
'semantically' these are different from an HTML markup / "semantics" point of view. from an end user point of view, they're interchangeable. At some point I should create an issue concerning these elements to see if the HTML editors would have any concerns with us also mapping b and i to strong and emphasis.

@scottaohara
Copy link
Member Author

Other no corresponding role elements, such as iframe, audio, video, etc. likely need some sort of role for implementors, but not for authors, if not one for both. but those would also need their own issues (which may already exist in ARIA). This is specifically for accName purposes to help determine what elements can be named.

To help, HTML AAM could also indicate on a per-element basis which elements are namable vs not in the mapping tables. This goes along with other updates that need to be done to the accName section to simplify/clarify that section of the spec.

@cookiecrook cookiecrook mentioned this issue Jun 30, 2022
Open
This was referenced Jul 1, 2022
Closed
Open
Open
Open
scottaohara added a commit that referenced this issue Jul 2, 2022
@scottaohara
Revise address element to map to role=group
a49381d 
Per discussion `<address>` in #373, WG had no objections to mapping the element to the `group` role.
This was referenced Jul 2, 2022
Merged
Merged
@scottaohara scottaohara added this to the ARIA 1.2 Related milestone Jul 8, 2022
This was referenced Jul 8, 2022
Merged
Open
scottaohara added a commit that referenced this issue Jul 9, 2022
@scottaohara
Mapping updates for generic elements
80a2d78 
related to #373
i, u, b, small map to generic.  Comments referring to the elements being exposed by their platform specific text styles remain in comments.

bdi/bdo map to generic. Reference writing-mode properties in comments for ia2/atk

`data` and `a no href` map to generic as decided.

also: simplifies br element mapping table
This was referenced Jul 9, 2022
Merged
Closed
scottaohara added a commit that referenced this issue Jul 9, 2022
@scottaohara
Mapping updates for generic elements (#426)
ccd0d4c 
related to #373
i, u, b, small map to generic.  Comments referring to the elements being exposed by their platform specific text styles remain in comments.

bdi/bdo map to generic. Reference writing-mode properties in comments for ia2/atk

`data` and `a no href` map to generic as decided.

also: simplifies br element mapping table
scottaohara added a commit that referenced this issue Jul 19, 2022
@scottaohara
Addition: Revise address element to map to role=group (#420)
4310e8d 
* Revise address element to map to role=group
* update changelog to ref date merged.
Per discussion `<address>` in #373, WG had no objections to mapping the element to the `group` role.
@cmorten cmorten mentioned this issue May 28, 2023
Closed
aarongable pushed a commit to chromium/chromium that referenced this issue Sep 8, 2023
@hs85jeong
[A11y] Update mapping for pre element
17e2b3c 
Update pre element to generic role according to the HTML-AAM spec[1] and
the discussion[2]. Therefore, the pre role was deprecated because it is
no longer used.

[1]: https://w3c.github.io/html-aam/#html-element-role-mappings
[2]: w3c/html-aam#373

Low-Coverage-Reason: Renaming changes only.
Bug: None
Change-Id: I7eaca4a966de762614751850787e1bcd154c8798
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4776721
Reviewed-by: Camille Lamy <[email protected]>
Commit-Queue: Hunseop Jeong <[email protected]>
Reviewed-by: David Tseng <[email protected]>
Reviewed-by: Aaron Leventhal <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1194019}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Element Mappings
Projects
None yet
Milestone
ARIA 1.2 Related
Development

Successfully merging a pull request may close this issue.

Mapping updates for generic elements w3c/html-aam
2 participants
@jnurthen @scottaohara