Closed Bug 480099 Opened 16 years ago Closed 16 years ago

If an html:label has both a title and inner text, title becomes acc name for control this label is labelling.

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
major

Tracking

()

VERIFIED FIXED

People

(Reporter: MarcoZ, Assigned: surkov)

References

(Blocks 1 open bug)

Details

(Keywords: access, regression)

Attachments

(2 files, 1 obsolete file)

Found in Bugzilla after bug 455886 landed:
If an HTML:label has both a @title attribute and inner text:
1. The inner text becomes the accName for the label as usual.
2. The contents of the @title attribute becomes the accName of the control this label is associated with via the @for attribute. This is a regression.
Attached file Testcase (deleted) —
This demonstrates the problem. Look at the accName of the label and the combobox, they should be identically called "Country:", but aren't.

The same can be observed in Bugzilla on any bug with the "Wanted-next" and other such blocking/wanted/etc. flags. And the factor is the title of the label rather than the title of the combobox, as in Bugzilla the same title is present on both controls. I reduced the testcase to the actual problematic constellation.
Attached patch mochitest (obsolete) (deleted) — Splinter Review
the problem is 
nsTextEquivUtils::AppendFromAccessible is called on label accessible and gets the label name. Since reqursive name from subtree is denied then we do not get name from subtree for label and get its name from @title attribute.
Blocks: namea11y
Attached patch patch (deleted) — Splinter Review
Assignee: nobody → surkov.alexander
Attachment #364079 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #364308 - Flags: review?(marco.zehe)
Attachment #364308 - Flags: review?(david.bolter)
Comment on attachment 364308 [details] [diff] [review]
patch

This fixes the bug, and on my system all tests pass. r=me for the functional part.
Attachment #364308 - Flags: review?(marco.zehe) → review+
Comment on attachment 364308 [details] [diff] [review]
patch

r=me
Thanks.
Attachment #364308 - Flags: review?(david.bolter) → review+
http://hg.mozilla.org/mozilla-central/rev/f2f97a49b6a1
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Flags: in-testsuite+
Verified fixed in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090228 Minefield/3.2a1pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: