Bug 109013 - AX: Some HTML native elements (header, section, footer, address) should have their own role descriptions instead of reusing the ARIA role descriptions (banner, region, contentinfo)
Summary: AX: Some HTML native elements (header, section, footer, address) should have ...
Status: ASSIGNED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P4 Normal
Assignee: James Craig
URL:
Keywords: InRadar
: 83989 109024 (view as bug list)
Depends on: 135116
Blocks:
  Show dependency treegraph
 
Reported: 2013-02-05 23:56 PST by James Craig
Modified: 2019-06-09 22:01 PDT (History)
6 users (show)

See Also:


Attachments
patch (12.85 KB, patch)
2013-04-18 00:26 PDT, James Craig
cfleizach: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description James Craig 2013-02-05 23:56:08 PST
<header> role description is 'banner' like [role="banner"]; we should add a new string for 'header'
Comment 1 James Craig 2013-02-05 23:59:39 PST
update role-subrole-description.html layout test when this is resolved.
Comment 2 James Craig 2013-02-06 00:45:13 PST
See also http://﷒0
Comment 3 James Craig 2013-02-06 00:45:51 PST
http://webkit.org/b/109024
Comment 4 Radar WebKit Bug Importer 2013-04-15 17:06:22 PDT
<rdar://problem/13658953>
Comment 5 James Craig 2013-04-17 23:41:59 PDT
*** Bug 109024 has been marked as a duplicate of this bug. ***
Comment 6 James Craig 2013-04-18 00:26:35 PDT
Created attachment 198688 [details]
patch
Comment 7 chris fleizach 2013-04-18 10:30:55 PDT
Comment on attachment 198688 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=198688&action=review

> Source/WebCore/English.lproj/Localizable.strings:688
> +/* accessibility role description for header element */

it seems confusing on the web to have a header and a heading. is there a difference?

> Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperMac.mm:1884
> +                if (m_object->renderer()->node()->hasTagName(headerTag))

the platform object shouldn't be reaching into the AX object like this. If this is a different role, then it should have a different AcccessibilityRole
Comment 8 James Craig 2013-04-18 22:52:25 PDT
(In reply to comment #7)
> (From update of attachment 198688 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=198688&action=review
> 
> > Source/WebCore/English.lproj/Localizable.strings:688
> > +/* accessibility role description for header element */
> 
> it seems confusing on the web to have a header and a heading. is there a difference?

One is a landmark container encompassing the header contents of the page and would be spoken as "header landmark." The other contains heading text of various levels and is used through the document, spoken as "heading level 2." 

> > Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperMac.mm:1884
> > +                if (m_object->renderer()->node()->hasTagName(headerTag))
> 
> the platform object shouldn't be reaching into the AX object like this.

Is there a better way to determine tagName than this? I would prefer we store tagName, id, and other relevant DOM attributes in the AX object (and potentially expose them to the APIs directly too), but we don't currently.

> If this is a different role, then it should have a different AcccessibilityRole

I disagree. This is merely for UI clarity. Functionally these are identical roles in that they are both landmarks and used almost entirely interchangeably. The only thing I'm changing is the localized role description, because for example, we know that the author has specifically chosen the <footer> tag, so it's more appropriate to describe the section to the user as "footer" than as the generic "content information" landmark.
Comment 9 James Craig 2013-08-16 10:47:04 PDT
What did we decide for this one? New webkit roles or special-cased role descriptions?
Comment 10 James Craig 2013-08-16 10:48:25 PDT
Looking back, it seems we're going with the "new webkit roles" approach.
Comment 11 James Craig 2013-12-20 11:46:05 PST
*** Bug 83989 has been marked as a duplicate of this bug. ***
Comment 12 Jason Kiss 2014-03-23 14:16:54 PDT
(In reply to comment #11)
> *** Bug 83989 has been marked as a duplicate of this bug. ***

I can see how bug 83989 is a dupe of this bug if the intention is to give <address> an AXRoleDescription of "address" instead of "content information", although I'm not sure I understand why it's called a landmark in the first place. 

Is it the intention that all "html native landmarks" get treated this way, or just <header> and <footer> (and <section>, which isn't a landmark per se)? 

Thanks.
Comment 13 James Craig 2014-12-08 19:40:18 PST
Yes. <address> too. Removed the word "landmark" from the bug title.