Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Comments

aside mapping revisions#350

Merged
stevefaulkner merged 7 commits intogh-pagesfrom
aside-mapping-updates
Apr 3, 2022
Merged

aside mapping revisions#350
stevefaulkner merged 7 commits intogh-pagesfrom
aside-mapping-updates

Conversation

@scottaohara
Copy link
Member

@scottaoharascottaohara commentedOct 29, 2021
edited
Loading

closes#86

Treat anaside element similarly toheader,footer andsection.

mapaside torole=complementary if:

  • scoped tomain orbody elements
  • or given accessible name if scoped to sectioning content or root elements

otherwise, map torole=generic


Preview |Diff


Preview |Diff

closes#86Treat an `aside` element similarly to `header`, `footer` and `section`.map `aside` to `role=complementary` if:* scoped to `main` or `body` elements* or given accessible name if scoped to sectioning content or root elementsotherwise, scope to `role=generic`
Copy link
Collaborator

@aleventhalaleventhal left a comment

Choose a reason for hiding this comment

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

Is there already consensus that it's a good idea?

I don't see any problems from an implementation point of view.

cookiecrook and scottaohara reacted with thumbs up emoji
@cookiecrook
Copy link
Collaborator

After a brief internal discussion, we think it’s a reasonable proposal, with a relatively simple implementation. Assuming no objections of merit arise, of course.

scottaohara reacted with thumbs up emoji

@scottaohara
Copy link
MemberAuthor

thanks@cookiecrook,@aleventhal

the tldr; intent of this update is to mitigate against the guidance from ARIA / the ARIA Authoring Practices (and conformance checkers that reference APG) that indicate that acomplementary landmark should generally be a top-level landmark, while the HTML specification indicates that use of an<aside> element can be far more permissive.

This change would allow for the<aside> to better handle both sets of authoring use, and then authors that specifically want to make sure an<aside> is exposed as a complementary landmark can still do so by giving it an accessible name, regardless of where it lives in the DOM. Otherwise, it allows authors to continue to use it without unknowingly overpopulating the number of complementary landmarks in the document.

@scottaohara
Copy link
MemberAuthor

test case created

JAWS-test reacted with thumbs up emoji

@spectranaut
Copy link
Contributor

spectranaut commentedMar 7, 2022
edited
Loading

ok@scottaohara I'm trying to implement this, and using your useful test case, and I ran into a bug in the conditional mapping of footer and header:https://bugs.chromium.org/p/chromium/issues/detail?id=1303917

I'm commenting here, though, because I think there is some lack of clarity in the spec. I couldn't find a definition of "sectioning element" and "sectioning root element"... are those phrases defined in some other spec? Or are they open to interpretation? If a sectioning element means a<section> (or<aside> or whatever the list of "sectioning elements" is) regardless of the ARIA role, for example, then your test case is wrong and the chrome implementation is correct.

Also is "scoped" defined somewhere? Regarding: "footer (scoped to thebody ormain element)". Trying to read this in order to implement it was difficult, is what I'm getting at. If "scoped" means "has a parent which is...", then a footer in some html might fall under both categories. For example, an<footer> inside an<article> inside of<main>. Could they be worded in a way that more obviously makes it mutually exclusive scenarios?

If neither of these things are defined anywhere, maybe this is the PR to add them? :)

@scottaohara
Copy link
MemberAuthor

scottaohara commentedMar 7, 2022
edited
Loading

@spectranautsectioning contnet andsectioning root are defined in HTML.

But to your point, yes, I think you're right and we need to define 'scoped' here to mitigate this question and help simplifyheader,footer and nowaside which all reference this.

and to be honest, we could probably even simplify that definition, as truly it's not just those elements... but if say adiv role=region were to interrupt the descendant relationship of thebody andaside, at the element level that would beok, because adiv isgeneric and doesn't fall into one of those categories. But by an author making it arole=region, thataside should no longer be arole=complementary.

EDIT: apologies. I realized when looking at this that it sure would have helped if I remembered to link to the HTML spec per those terms. Those links have been added in.

JAWS-test and spectranaut reacted with thumbs up emoji

adds links to sectioning content and sectioning root, as was done in `header` and `footer`
@spectranaut
Copy link
Contributor

Awesome thanks for adding the links, that helps!

I'm not sure what you are suggesting exactly for simpler categories, but I was just thinking something like:

  1. Aside (scoped to a sectioning content element, or a sectioning root element other thanbody)
  2. Aside (not scoped to a sectioning content element, or a sectioning root element other thanbody)

I still don't know if "scoped" is the right word or where the definition for it should go, or if it is self-evident to people other than me?

@scottaohara
Copy link
MemberAuthor

Since it's not an officially defined term as far as i can tell, i'm not opposed to changing it to another word(s) if that makes the most sense. There are other similar terms in ARIA and HTML that sorta fit this concept....

anyway, sent you an email to try and discuss further outside of back and forth issue comments. if that doesn't work for you though, happy to keep at this until we get this sorted.

@stevefaulknerstevefaulkner merged commit8b8d38c intogh-pagesApr 3, 2022
@scottaoharascottaohara deleted the aside-mapping-updates branchApril 5, 2022 15:16
scottaohara added a commit to w3c/aria that referenced this pull requestAug 18, 2022
resolves#1396Clarifies that complementary content at a 'similar level' could be a sibling to the main content, or a direct child of the main content.  This better aligns with the permissiveness of HTML's `<aside>` element which has no requirements of not being allowed within a `<main>`. Related:w3c/html-aam#350
pkra pushed a commit to w3c/aria that referenced this pull requestAug 25, 2022
resolves#1396Clarifies that complementary content at a 'similar level' could be a sibling to the main content, or a direct child of the main content.  This better aligns with the permissiveness of HTML's `<aside>` element which has no requirements of not being allowed within a `<main>`. Related:w3c/html-aam#350
pkra pushed a commit to w3c/aria that referenced this pull requestAug 25, 2022
resolves#1396Clarifies that complementary content at a 'similar level' could be a sibling to the main content, or a direct child of the main content.  This better aligns with the permissiveness of HTML's `<aside>` element which has no requirements of not being allowed within a `<main>`. Related:w3c/html-aam#350
jnurthen pushed a commit to w3c/aria that referenced this pull requestOct 10, 2023
resolves#1396Clarifies that complementary content at a 'similar level' could be a sibling to the main content, or a direct child of the main content.  This better aligns with the permissiveness of HTML's `<aside>` element which has no requirements of not being allowed within a `<main>`. Related:w3c/html-aam#350
Copy link
Collaborator

@cookiecrookcookiecrook left a comment

Choose a reason for hiding this comment

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

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@cookiecrookcookiecrookcookiecrook left review comments

@aleventhalaleventhalaleventhal left review comments

@stevefaulknerstevefaulknerstevefaulkner approved these changes

@joanmariejoanmarieAwaiting requested review from joanmarie

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Suggest <aside> should map to complementary with restrictions

5 participants

@scottaohara@cookiecrook@spectranaut@stevefaulkner@aleventhal

[8]ページ先頭

©2009-2026 Movatter.jp