My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 2866: Lost function in new Gerrit GUI: Browse files in patch set
8 people starred this issue and may be notified of changes. Back to list
Status:  New
Owner:  ----


Sign in to add a comment
 
Reported by yzhang99...@gmail.com, Aug 26, 2014
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
Just press 'u' and you can see the file list on the change screen.
Aug 26, 2014
#2 yzhang99...@gmail.com
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
#3 sieb...@gmail.com
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
#4 patrick....@gmail.com
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
#5 yzhang99...@gmail.com
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

Powered by Google Project Hosting