Closed Bug 402597 Opened 17 years ago Closed 17 years ago

AT-SPI Accessibility:Event.user_data None for ARIA tooltips/alerts

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: scott, Assigned: ginnchen+exoracle)

References

(Blocks 1 open bug)

Details

(Keywords: access, Whiteboard: orca:normal)

Attachments

(1 file)

The tooltip seen here http://www.mozilla.org/access/dhtml/new/tooltip has the following debug for the Accessibility::Event object. Note that the any_data field, the target of the object:children-changed:add event, is None. source = [paragraph | ] type = object:children-changed:add:system any_data = None detail1 = 2 detail2 = 0 This is also the case for dojo tooltips (actually alerts) seen here http://archive.dojotoolkit.org/nightly/dojotoolkit/dijit/tests/test_Tooltip.html and the tooltips linked from http://developer.mozilla.org/en/docs/Accessible_DHTML#Supported_roles . A strange oddity is that object:children-changed:add events for the live region examples were broken (pre-pyatspi migration for Orca) with respect to user_data, but now they work. Beyond event type, I am not sure how else the two issues are related.
Accessibility::Event.any_data is correct if you change the 'span' to a 'p' for the tooltip in http://www.mozilla.org/access/dhtml/new/tooltip . Please fix span issue (tough one I'm sure) or change the example, possibly mentioning <span> unsupported. Now investigating why UIUC and dojo examples are unsupported with regard to any_data.
The dojo example seen here http://archive.dojotoolkit.org/nightly/dojotoolkit/dijit/tests/test_Tooltip.html also uses a span for the tooltip. However, changing it to a <p> did not fix the problem. I tried several markup changes to the surrounding tooltip markup, all with no luck. The event.any_data field is always None for object:children-changed:add:system events. Also of note is that the object:text-changed:inserted:system events appear to be of little worth to Orca for tooltip announcements.
See also bug 402049.
Blocks: aria, fox3access
Keywords: access
OS: Linux → All
Hardware: PC → All
The UIUC tooltip examples can be fixed by changing their hiding/showing logic to use display:[none|inline] instead of moving the tooltip way off the screen.
Whiteboard: orca:normal
We should not use accessible->GetIndexInParent(&indexInParent); in nsAccessibleWrap::FireAtkShowHideEvent because for ATK we don't want to include text leaf nodes as children.
Assignee: aaronleventhal → ginn.chen
Attached patch patch (deleted) — Splinter Review
reuse the code above
Attachment #299979 - Flags: review?(aaronleventhal)
Comment on attachment 299979 [details] [diff] [review] patch Nice work Ginn! I bet this solves a lot of other issues with show/hide events!
Attachment #299979 - Flags: review?(aaronleventhal)
Attachment #299979 - Flags: review+
Attachment #299979 - Flags: approval1.9?
Comment on attachment 299979 [details] [diff] [review] patch a1.9+=damons
Attachment #299979 - Flags: approval1.9? → approval1.9+
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Checked in for Ginn.
This bug and it's sister bug #402049 are both looking much better.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: