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 510478 - Character navigation in FF wraps from bottom of file to top
Character navigation in FF wraps from bottom of file to top
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
2.21.x
Other All
: Normal normal
: 2.22.0
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
Depends on:
Blocks: 404403
 
 
Reported: 2008-01-18 20:59 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2008-07-22 19:33 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
revision 1 (1.18 KB, patch)
2008-01-26 07:55 UTC, Joanmarie Diggs (IRC: joanie)
none Details | Review
revision 2 (1.53 KB, patch)
2008-01-27 01:23 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2008-01-18 20:59:16 UTC
As originally reported by Jon:
http://bugzilla.gnome.org/show_bug.cgi?id=509482#c4

Arrowing right beyond the last line of webpage cycles up to start. At start arrowing left does not take the user to bottom.
Comment 1 Joanmarie Diggs (IRC: joanie) 2008-01-26 07:55:37 UTC
Created attachment 103768 [details] [review]
revision 1

The question is, what do we do/say when we get to the very end?  And for that matter, even though we don't wrap to the bottom from the top, we don't say anything either.  So, should we:

1. Say nothing at all and allow the user to infer from the silence
   where they are/that they can't go even further?

2. Say "Top of File" at the top and "End of File" at the bottom?

3. Just repeat the current character, which tells the user that their
   keystroke is being acknowledged and processed.

4. Something else?

Mike, I need your input.

In the meantime, revision 1 of this patch stops the wrapping (i.e. fixes the bug) and uses approach 3 for feedback to the user.  Approach 1 and 2 should be just about as easy.  Let me know.  Thanks.

Pylinted already.  The regression tests take awhile to run and it's nearly 3 AM, so I'll start the tests and go to bed. <smile>  But to be honest, our regression tests don't do very much testing w.r.t. character-by-character navigation.  I need to add more tests on that front....
Comment 2 Joanmarie Diggs (IRC: joanie) 2008-01-26 08:01:37 UTC
Jon, I'm adding you to the CC list.  This bug is what you reported on bug 509482:

> Sorry, maybe it should come under a diffrent bug, but here it is
> for now: arrowing right beyond the last line of webpage  cycles 
> up to start. at start arrowing left does not take the user to 
> bottom.

Please give the attached patch a try.  Also, I'm interested in your feedback on the "what should we say" question I asked of Mike in my previous comment.  Thanks much!!

P.S. Guys, I'd be super happy if we can get this in for 2.21.90. <smile>  If y'all are around this weekend and bored and want to pass the time testing.... <grin>
Comment 3 Mesar Hameed 2008-01-26 12:13:06 UTC
Joanie, Bug fixed, thanks!

regarding user feedback:
either alternative 1 or 3 is fine, although for personal preference 1 is more likeable.
If i wish to get to top of page, i keep up arrow pressed, if it silences then i know i've arrived.
If "top of page" is echoed, i may have held the key for too long and the message gets repeated, which would be frustrating, because i would then want to be able to move on immediately.

Mike, what is your feeling on this?
Comment 4 Mike Pedersen 2008-01-26 18:58:01 UTC
I personally prefer option 1 because it follows what we do in applications such as gedit.  
Comment 5 Joanmarie Diggs (IRC: joanie) 2008-01-26 19:06:12 UTC
Okey dokey guys.  Thanks!  New patch in a little bit.
Comment 6 Joanmarie Diggs (IRC: joanie) 2008-01-27 01:23:56 UTC
Created attachment 103796 [details] [review]
revision 2

Please test.  Thanks.
Comment 7 Mike Pedersen 2008-01-27 01:43:49 UTC
Works great.  thanks 
Comment 8 Joanmarie Diggs (IRC: joanie) 2008-01-27 02:43:51 UTC
Thanks Mike.  Patch committed.  Moving to pending.
Comment 9 Mike Pedersen 2008-01-28 17:50:45 UTC
works great.  thanks 
Comment 10 Joanmarie Diggs (IRC: joanie) 2008-01-28 19:47:41 UTC
Thanks Mike.  Closing as FIXED.