New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 7815 link

Starred by 3 users

Issue metadata

Status: Released
Last visit 17 days ago
Closed: Dec 2017
ReleasedIn: 422.0

Sign in to add a comment

Unbind Up/Down from navigating the file list.

Project Member Reported by, Nov 21 2017

Issue description

What steps will reproduce the problem?
1. Open a code review
2. Use Up/Down keys

What is the expected output?

The page moves up and down

What do you see instead?

Nothing. It doesn't work. Or worse, it will move up and down the file list, jumping from wherever I happen to be on the page.

Also, when diffs are open and I'm trying to review code, I'll often instinctively use Up/Down or j/k to try and move around a small increment (unlike Page Up/Page Down, which move a lot). It would be nice if j/k kept their current mapping and Up/Down were unbound so they did what you would expect them to on an ordinary page. :)
Project Member

Comment 1 by, Nov 24 2017

Components: PolyGerrit
Project Member

Comment 2 by, Nov 27 2017

Labels: -Priority-3 Hotlist-FileList Triaged-Yes Priority-2
Status: Accepted (was: New)
Project Member

Comment 3 by, Nov 29 2017

Status: ChangeUnderReview (was: Accepted)
Project Member

Comment 4 by, Dec 5 2017

ReleasedIn: 422.0
Status: Released (was: ChangeUnderReview)
Thanks very much!
Project Member

Comment 6 by, Mar 19 2018

 Issue 8563  has been merged into this issue.
Labels: FixedIn-2.15.2
Status: Submitted (was: Released)
Accidentally set to released because a search for label:FixedIn-2.15 also includes the ones with FixedIn-2.15.2
Project Member

Comment 9 by, Apr 11 2018

Status: Released (was: Submitted)
Project Member

Comment 10 by, Nov 20

Labels: FixedIn-2.16

Sign in to add a comment