Closed
Bug 294222
Opened 19 years ago
Closed 19 years ago
Fire MSAA scrolling_start for anchor jumps, not selection_within
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: aaronlev)
Details
(Keywords: access)
Attachments
(1 file)
2.06 KB,
patch
|
timeless
:
review+
neil
:
superreview+
shaver
:
approval1.8b3+
|
Details | Diff | Splinter Review |
We're going to be using selection_within for what it was originally intended for -- it gets fired on widgets capable of multiple selection when the selection within it changes. This bug is to use the MSAA scrolling_end event as our anchor jump indication.
Assignee | ||
Comment 1•19 years ago
|
||
Actually it will be better to use scrolling_start, so that we don't have dual meanings for scrolling_end. Right now we're using scrolling_end for any kind of scroll occurance.
Summary: Fire MSAA scrolling_end for anchor jumps, not selection_within → Fire MSAA scrolling_start for anchor jumps, not selection_within
Assignee | ||
Comment 2•19 years ago
|
||
Attachment #183619 -
Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #183619 -
Flags: review?(timeless)
Comment on attachment 183619 [details] [diff] [review] selection_within -> scrolling_start >Index: accessible/src/msaa/nsDocAccessibleWrap.cpp >@@ -278,11 +278,11 @@ nsDocAccessibleWrap::GetFirstLeafAccessi >+ // Fire SCROLLING_START on first leaf accessible -- because some > // assistive technologies only cache the child numbers for leaf accessibles > // the can only relate events back to their internal model if it's a leaf. they? :) > // There is usually an accessible for the focus node, but if it's an empty text node > // we have to move forward in the document to get one
Attachment #183619 -
Flags: review?(timeless) → review+
Updated•19 years ago
|
Attachment #183619 -
Attachment description: selection_within -> scrolling_end → selection_within -> scrolling_start
Attachment #183619 -
Flags: superreview?(neil.parkwaycc.co.uk) → superreview+
Assignee | ||
Updated•19 years ago
|
Attachment #183619 -
Flags: approval1.8b3?
Comment on attachment 183619 [details] [diff] [review] selection_within -> scrolling_start a=shaver
Attachment #183619 -
Flags: approval1.8b3? → approval1.8b3+
Assignee | ||
Comment 5•19 years ago
|
||
Checking in accessible/src/msaa/nsDocAccessibleWrap.cpp; /cvsroot/mozilla/accessible/src/msaa/nsDocAccessibleWrap.cpp,v <-- nsDocAccessibleWrap.cpp new revision: 1.27; previous revision: 1.26 done
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•