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 567984 - Structural navigation needs to consider text within the document frame
Structural navigation needs to consider text within the document frame
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
2.25.x
Other All
: Normal normal
: 2.24.4
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
Depends on:
Blocks: 404403
 
 
Reported: 2009-01-16 15:13 UTC by Hammer Attila
Modified: 2009-01-21 22:45 UTC
See Also:
GNOME target: ---
GNOME version: 2.23/2.24


Attachments
revision 1 (plus a new regression test) (6.64 KB, patch)
2009-01-18 05:11 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review
additional fix (3.79 KB, patch)
2009-01-21 02:33 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Hammer Attila 2009-01-16 15:13:53 UTC
Please describe the problem:

Dear developers!

When trying navigating heading with the structural navigating commands and
press after the navigation one or more left arrow and after this pressing
heading structural navigation commands (for example h jump to next heading),
Orca jumping the first heading not a next heading.
I seeing this problem for example with www.index.hu/vakbarat webpage.

Jon confirm this problem with bug561660 bugreport comment.

Attila

Steps to reproduce:
1. Open for example with www.index.hu/vakbarat webpage.
2. Try navigate headings for example with h key, and after some navigation press down arrow (one or two down arrow).
3. Press again the h key. If you like, orca jumping a first heading, not a next heading.


Actual results:


Expected results:


Does this happen every time?
Yes.

Other information:
Comment 1 Willie Walker 2009-01-16 15:21:38 UTC
Confirmed on OpenSolaris 20081.11 with Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv:1.9.0.4) Gecko/2008111710 Firefox/3.0.4.

Very strange since this seems like a very simple web page.
Comment 2 Joanmarie Diggs (IRC: joanie) 2009-01-18 00:54:40 UTC
Seems to be related to the collection interface. If I set orca.settings.useCollection to False, the problem goes away. Assigning this one to myself. Stay tuned.
Comment 3 Joanmarie Diggs (IRC: joanie) 2009-01-18 05:11:40 UTC
Created attachment 126672 [details] [review]
revision 1 (plus a new regression test)

This solves the problem for me. I've also created a new regression test based upon this issue. Please test. Thanks!
Comment 4 Hammer Attila 2009-01-18 10:23:23 UTC
Thanks Joanmarie, I will be testing the patch monday afternoon.

Attila
Comment 5 Hammer Attila 2009-01-19 14:39:50 UTC
Joanmarie, testing the patch.
The h navigation key and heading navigation numbers work, But the Shift+h key and shift+heading navigation numbers (1-6) is not work good.
If would like, try this steps:
1. Launch Firefox, and open www.index.hu/vakbarat webpage.
2. Navigate any heading, and press one or more down arrow key.
3. Press shift+h or shift+heading navigation numbers, the structural navigation jumps bottom of the webpage, and jumps the last heading, not previous heading.

If need, I attaching debug.out file.

Attila
Comment 6 Joanmarie Diggs (IRC: joanie) 2009-01-19 16:36:00 UTC
Thanks for testing Attila! Confirmed. :-( I've also confirmed that the issue you raise in comment #5 occurs without the patch and even when orca.settings.useCollection is set to False. (i.e. it's a different issue). Therefore I've committed the current patch to trunk as a "partial fix" so that it gets included for the 2.25.5 release. I'll see what's up with navigating by previous object/heading.
Comment 7 Joanmarie Diggs (IRC: joanie) 2009-01-21 02:33:27 UTC
Created attachment 126898 [details] [review]
additional fix

Patch plus updated regression test.

Pylinted and regression tested for all the structural-navigation-related Firefox tests for both FF 3.0.5 and 3.1b2. Please test.

(Given that this patch is limited to structural_navigation.py, the full firefox suite of tests should not show any regressions, but I am extremely paranoid so I'll run those this evening to be sure.) :-)
Comment 8 Joanmarie Diggs (IRC: joanie) 2009-01-21 07:48:12 UTC
The patch has been fully regression tested using the Firefox suite for 3.0.5 and 3.1b2 in OpenSolaris. Given that, and the pylinting, and the fact that it's limited to structural navigation I've gone ahead and committed the patch to trunk.

Attila, as a result, you don't need a patch to test the changes; just the latest trunk. Thanks for all your help!!

Moving to pending.
Comment 9 Willie Walker 2009-01-21 15:07:38 UTC
The simplicity and safeness of the patch, along with the good impact it has, seems to make this a candidate for gnome-2-24.  Thoughts?
Comment 10 Joanmarie Diggs (IRC: joanie) 2009-01-21 22:45:43 UTC
Agreed. Both patches have been committed to the gnome-2-24 branch.

Closing as FIXED.