Closed Bug 526915 Opened 10 years ago Closed 10 years ago

aria-describedby broken in FF 3.6b1

Categories

(Core :: Disability Access APIs, defect)

x86
Windows XP
defect
Not set

Tracking

()

VERIFIED FIXED
Tracking Status
status1.9.2 --- beta3-fixed

People

(Reporter: hans.hillen, Assigned: surkov)

Details

(Keywords: regression, verified1.9.2)

Attachments

(2 files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2b1) Gecko/20091029 Firefox/3.6b1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2b1) Gecko/20091029 Firefox/3.6b1

Descriptive text added to an element using aria-describedby is not exposed as MSAA description.

Reproducible: Always

Steps to Reproduce:
Use attached sample. It contains a non-functional ARIA button and text that describes the button. 
Tab to the button so that it receives focus.
Actual Results:  
Descriptive text is not announced by NVDA and  the 'description' field is empty in MSAA inspect tools. Interestingly enough, JAWS 11 still reads the description despite it not being exposed in MSAA

Expected Results:  
The descriptive text target with aria-describedby is exposed as MSAA description, as it did in FF3.5
Component: Disability Access → Disability Access APIs
Product: Firefox → Core
QA Contact: disability.access → accessibility-apis
Marco, if possible I'd like a regression window.
This is confirmed for this specific testcase. However, my original Easy ARIA Tip #2 from http://www.marcozehe.de/2008/03/23/easy-aria-tip-2-aria-labelledby-and-aria-describedby/ still works, so this is something specific to this testcase.
Status: UNCONFIRMED → NEW
Ever confirmed: true
This was introduced in the following build:
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090219 Minefield/3.2a1pre (.NET CLR 3.5.30729)
The build from February 18 is still OK.
Attached patch patchSplinter Review
Assignee: nobody → surkov.alexander
Status: NEW → ASSIGNED
Attachment #410734 - Attachment is patch: true
Attachment #410734 - Attachment mime type: application/octet-stream → text/plain
Attachment #410734 - Flags: review?(marco.zehe)
Attachment #410734 - Flags: review?(bolterbugz)
Comment on attachment 410734 [details] [diff] [review]
patch

r=me. Thanks!
Attachment #410734 - Flags: review?(marco.zehe) → review+
(In reply to comment #6)
> (From update of attachment 410734 [details] [diff] [review])
> r=me. Thanks!

I'm really going to love our name rules map, it's getting pretty easy to fix bugs :)
Comment on attachment 410734 [details] [diff] [review]
patch

r=me, nice to see the name rules pay off :)
Attachment #410734 - Flags: review?(bolterbugz) → review+
landed on 1.9.3 - http://hg.mozilla.org/mozilla-central/rev/b697b641e508
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
This is a regression from Firefox 3.5 and will impact all web authors trying to USE aria-describedby with certain constructs.
Flags: blocking1.9.2?
Keywords: regression
Attachment #410734 - Flags: approval1.9.2?
Comment on attachment 410734 [details] [diff] [review]
patch

Safe fix, has tests.
Verified fixed in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.3a1pre) Gecko/20091110 Minefield/3.7a1pre (.NET CLR 3.5.30729)
Status: RESOLVED → VERIFIED
Flags: blocking1.9.2? → blocking1.9.2+
(In reply to comment #13)
> Pushed to 1.9.2 on Alexander's behalf:
> http://hg.mozilla.org/releases/mozilla-1.9.2/rev/b0c3db90958d

Thank you!
Verified fixed in Mozilla/5.0 (Windows; U; Windows NT 6.1; de; rv:1.9.2b3) Gecko/20091115 Firefox/3.6b3 (.NET CLR 3.5.30729)
Keywords: verified1.9.2
You need to log in before you can comment on or make changes to this bug.