Closed
Bug 340672
Opened 18 years ago
Closed 18 years ago
New ATK: Expose Value Change events
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: aaronlev)
References
Details
(Keywords: access)
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
evan.yan
:
review+
|
Details | Diff | Splinter Review |
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 | ||
Updated•18 years ago
|
Assignee: aaronleventhal → gaomingcn
Assignee | ||
Updated•18 years ago
|
Assignee: gaomingcn → aaronleventhal
Assignee | ||
Updated•18 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•18 years ago
|
||
Assignee | ||
Updated•18 years ago
|
Attachment #226688 -
Attachment is obsolete: true
Assignee | ||
Comment 2•18 years ago
|
||
Would like to test with http://www.mozilla.org/access/dhtml/slider
Attachment #226689 -
Flags: review?(ginn.chen)
Assignee | ||
Comment 3•18 years ago
|
||
Attachment #226689 -
Attachment is obsolete: true
Attachment #226690 -
Flags: review?(ginn.chen)
Attachment #226689 -
Flags: review?(ginn.chen)
Assignee | ||
Updated•18 years ago
|
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+
Assignee | ||
Updated•18 years ago
|
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.
Description
•