gco / rietveld

Automatically exported from code.google.com/p/rietveld
Apache License 2.0
0 stars 0 forks source link

Up and down arrow no longer scroll the diff page as expected #378

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Press down arrow
2. A highlight line appears in the code area, rather than scrolling the window

What is the expected output? What do you see instead?
I expect the browser to scroll the window down

What browser are you using?  What version? On what operating system?
Firefox 12.0 OSX

At what URL are you accessing Rietveld?  (e.g. codereview.appspot.com)
Please note if you are using the Google Apps Labs version (e.g.
codereview.<yourdomain>).
http://cr.bittorrent.com/7870014/diff/10001/bittorrent.cpp

*** If you are a Google employee please say so or mail me (Guido van
Rossum) directly. ***

Please provide any additional information below.
It'd be nice if users could turn the highlight line off and just have arrows be 
arrows.

Original issue reported on code.google.com by art.yer...@gmail.com on 8 May 2012 at 6:02

GoogleCodeExporter commented 9 years ago
This change was intentional (see revision 140baeb01745) to allow line by line 
navigation and addin comments by pressing "Enter" on every line. Why do you 
think that this behavior differs so much from the default browser behavior?

Original comment by albrecht.andi on 8 May 2012 at 7:06

GoogleCodeExporter commented 9 years ago
Working as intended.

Original comment by gvanrossum@gmail.com on 8 May 2012 at 7:40

GoogleCodeExporter commented 9 years ago
Because I use the arrow keys to scroll the page of diffs as a whole.  Now
in order to get the first scroll down, I have to scroll below the entire
first page, one line at a time.  Also, this skips over blocks of
commentary, which could be quite long.  Perhaps I'm reading a long block of
reviewer's comments and want to scroll through them.

Also, the scrolling, when the highlight line moves off the screen, is emacs
style half-page at a time, whereas the arrow keys generally scroll 2 or 3
lines at a time, depending on text size.

Maybe I'm in the minority, but I think it's kind of jarring when a web page
takes an action on one of the standard keys on the keyboard.  It'd be
different if the highlight line could be selected with the mouse, reacted
to pageup and pagedown, allowed focusing and scrolling through commentary,
and reacted to the overall page scrollbars on the document, as that'd at
least work as expected.

Here's an experiment: Press Up or down once, scroll to the bottom of a diff
page using the browser scroll bar, now press up again.  I think you can
easily observe that it isn't the same as in a normal web page.

Original comment by art.yer...@gmail.com on 8 May 2012 at 7:58

GoogleCodeExporter commented 9 years ago
We can't make everyone happy, sorry.

Original comment by gvanrossum@gmail.com on 8 May 2012 at 8:30

GoogleCodeExporter commented 9 years ago
You encouraged me to try out noscript, so it's not a total loss, I suppose.

Original comment by art.yer...@gmail.com on 8 May 2012 at 8:44

GoogleCodeExporter commented 9 years ago
Issue 380 has been merged into this issue.

Original comment by gvanrossum@gmail.com on 11 May 2012 at 11:09

GoogleCodeExporter commented 9 years ago
You could make most people happy by using a different set of keys for the line 
by line navigation and leaving arrow keys as they should be.

Original comment by mattm@chromium.org on 11 May 2012 at 11:17

GoogleCodeExporter commented 9 years ago
That would make most current users unhappy though. And no, I'm not going to add 
a config option.

Original comment by gvanrossum@gmail.com on 11 May 2012 at 11:29

GoogleCodeExporter commented 9 years ago
I'd say we - open two issues and propose people to vote for it.

Original comment by techtonik@gmail.com on 12 May 2012 at 5:33