After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 566073 - Orca provides the "tree level" for items in submenus within Firefox 3.x
Orca provides the "tree level" for items in submenus within Firefox 3.x
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: speech
2.25.x
Other All
: Normal minor
: 2.24.3
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
Depends on:
Blocks: 404403
 
 
Reported: 2008-12-30 19:34 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2009-01-05 19:15 UTC
See Also:
GNOME target: ---
GNOME version: 2.23/2.24


Attachments
fix plus regression test updates (8.56 KB, patch)
2008-12-30 19:47 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2008-12-30 19:34:04 UTC
When one first arrows into a Firefox submenu (e.g. Page Style, Toolbars), Orca speaks the tree level associated with the item which received focus as a result. The item which received focus is, of course, not in a tree and therefore a level should not be spoken.

The reason for this bug has to do with the way we're getting the node level in the Gecko script, namely by looking for the attribute "level". In addition to items within trees, this attribute is present for headings (currently checked for) and -- it turns out -- items in submenus. :-)  We need to check for that condition as well.
Comment 1 Joanmarie Diggs (IRC: joanie) 2008-12-30 19:47:58 UTC
Created attachment 125547 [details] [review]
fix plus regression test updates

The fix is a one-liner: Check to see if the parent of the item in question is of ROLE_MENU. If it is, we're not in a tree.

This patch also includes updates to the regression tests, mostly removing the "BUG?" lines associated with this bug, but also a bit more work to make the tests be usable for both Firefox 3.0.5 and Firefox 3.1beta2. The fix has been regression tested for both versions of Firefox, and (of course) pylinted. :-)

Given that this is another very limited change, I've gone ahead and committed this to trunk. I realize that I've done this a few times now. It won't become my modus operandi, honest. :-) We just need to get the tests in shape and the low-hanging (ground-resting) fruit out of the way so that we can do proper testing of the more significant patches awaiting feedback.
Comment 2 Willie Walker 2009-01-05 13:42:03 UTC
This seems safe and appropriate for gnome-2-24 as well.  Joanie -- if you agree, please check it into gnome-2-24.
Comment 3 Joanmarie Diggs (IRC: joanie) 2009-01-05 19:15:27 UTC
Done. The regression tests are so different now that I didn't commit the test changes; just the fix. :-)

Readjusting the milestone.