| Issue 2866: | Lost function in new Gerrit GUI: Browse files in patch set | |
| 7 people starred this issue and may be notified of changes. | Back to list |
In the old review, I can use shortcut "f" to navigate among files while code review. This is a very important feature for me do an effective code review. However, this feature somehow is took out in the new GUI. Could you please put it back? Best Regards, Yi
Aug 26, 2014
Project Member
#1
edwin.ke...@gmail.com
Aug 26, 2014
Thank you for your reply! I guess I can do that. The reason I don't use "U" is that there is one more page-reloading and sometimes is slow.
Jan 21, 2015
The navigation screen that has been removed in the new screen is regression if you'd ask me. It's a nifty feature that can save some time. Switching back and forth between the change overview and the diff is clumsy and in some cases fairly slow. As a workaround I usually open two tabs in my browser, one showing the diff and one showing the change overview. This is merely a workaround, not a proper fix.
Nov 16, 2015
Also missing the quick file list, 'u' is my fix for it but every time I use it I'm annoyed about how slow it is... Sometimes I'm not even interested in changing file, I just want to check the name of another file in the patch set, and when that's the case it really is a bummer to have to reload two pages.
Nov 16, 2015
This function provides me a quick code review context and navigation so I can review files in a logic order. Without this function, I am using two windows, switch back and forth so I don't have to wait for loading. |
|
| ► Sign in to add a comment |