The default bug view has changed. See this FAQ.

New ATK: Expose Value Change events

RESOLVED FIXED

Status

()

Core
Disability Access APIs
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: Aaron Leventhal, Assigned: Aaron Leventhal)

Tracking

({access})

Trunk
x86
Linux
access
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

11 years ago
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

11 years ago
Assignee: aaronleventhal → gaomingcn
(Assignee)

Updated

11 years ago
Assignee: gaomingcn → aaronleventhal
(Assignee)

Updated

11 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 1

11 years ago
Created attachment 226688 [details] [diff] [review]
Simple patch
(Assignee)

Updated

11 years ago
Attachment #226688 - Attachment is obsolete: true
(Assignee)

Comment 2

11 years ago
Created attachment 226689 [details] [diff] [review]
Having trouble testing this because of AT poke freezes, but looks right

Would like to test with http://www.mozilla.org/access/dhtml/slider
Attachment #226689 - Flags: review?(ginn.chen)
(Assignee)

Comment 3

11 years ago
Created attachment 226690 [details] [diff] [review]
Having trouble testing this because of AT poke freezes, but looks right
Attachment #226689 - Attachment is obsolete: true
Attachment #226690 - Flags: review?(ginn.chen)
Attachment #226689 - Flags: review?(ginn.chen)
(Assignee)

Updated

11 years ago
Attachment #226690 - Flags: review?(ginn.chen) → review?(Evan.Yan)

Comment 4

11 years ago
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

11 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.