613502 - Map <article> like we do aria role "article"

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[David Bolter [:davidb] (NeedInfo me for attention)]

Spun off bug 610650. Users normally navigate from heading to heading anyway. Exposing article as a landmark doesn't gain us anything.

Comment 1

[David Bolter [:davidb] (NeedInfo me for attention)]

So role:ROLE_DOCUMENT
Default state:STATE_READONLY

Comment 2

[David Bolter [:davidb] (NeedInfo me for attention)]

Attachment: patch

Our readonly state setting is sorta convoluted in general. We probably want a clean up bug for it.

Comment 3

[alexander :surkov (:asurkov)]

Comment on attachment 491921 [details] [diff] [review]
patch


>+  } else if (mContent && mContent->Tag() == nsAccessibilityAtoms::article) {

you don't need to check mContent

>+    // we want <article> to behave like a document in terms of readonly state

Use capital letter and dot in the end.

>     SimpleTest.waitForExplicitFinish();
>     addA11yLoadEvent(doTest);
>@@ -52,12 +57,18 @@
>     title="Expose non-editable documents as readonly, regardless of role">
>      Mozilla Bug 467387
>   </a>

add link to the bug

>   <p id="display"></p>
>   <div id="content" style="display: none"></div>
>   <pre id="test">
>   </pre>

and here

in the future we'd like to share nsARIAMap for ARIA and HTML5 I think

Comment 4

[Marco Zehe (:MarcoZ)]

Comment on attachment 491921 [details] [diff] [review]
patch

>+  <article id="article">an article</article>
>   <article id="document" role="document">a document</article>

Maybe morph this into a test for landmark role "main", otherwise the ARIA override is no longer a true override.

Comment 5

[David Bolter [:davidb] (NeedInfo me for attention)]

(In reply to comment #3)

> in the future we'd like to share nsARIAMap for ARIA and HTML5 I think

Yes, and possibly rename it.

Comment 6

[David Bolter [:davidb] (NeedInfo me for attention)]

http://hg.mozilla.org/mozilla-central/rev/d0a46bfe92d5 (a=davidb)