Closed Bug 340672 Opened 18 years ago Closed 18 years ago

New ATK: Expose Value Change events

Categories

(Core :: Disability Access APIs, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: aaronlev, Assigned: aaronlev)

References

Details

(Keywords: access)

Attachments

(1 file, 2 obsolete files)

We currently don't expose value change events, although they are very important for support of progress meters and sliders: We don't expose these events because it was too difficult to find the old value. Let's find a way to expose the event without providing the old value. The spec does not say that detail1 has to be the old value. See the code: http://lxr.mozilla.org/seamonkey/source/accessible/src/base/nsRootAccessible.cpp#83 837 #if 0 838 // XXX todo: value change events for ATK are done with 839 // AtkPropertyChange, PROP_VALUE. Need the old and new value. 840 // Not sure how we'll get the old value. 841 // Aaron: I think this is a problem with the ATK API -- its much harder to 842 // grab the old value for all the application developers than it is for 843 // AT's to cache old values when they need to (when would that be!?) 844 else if (eventType.LowerCaseEqualsLiteral("valuechange")) { 845 privAcc->FireToolkitEvent(nsIAccessibleEvent::EVENT_VALUE_CHANGE, 846 accessible, nsnull); 847 } 848 #endif
Assignee: aaronleventhal → gaomingcn
Assignee: gaomingcn → aaronleventhal
Status: NEW → ASSIGNED
Attached patch Simple patch (obsolete) (deleted) — Splinter Review
Attachment #226688 - Attachment is obsolete: true
Attachment #226689 - Flags: review?(ginn.chen)
Attachment #226689 - Attachment is obsolete: true
Attachment #226690 - Flags: review?(ginn.chen)
Attachment #226689 - Flags: review?(ginn.chen)
Attachment #226690 - Flags: review?(ginn.chen) → review?(Evan.Yan)
Comment on attachment 226690 [details] [diff] [review] Having trouble testing this because of AT poke freezes, but looks right r=me
Attachment #226690 - Flags: review?(Evan.Yan) → review+
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: