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

Issue 661189 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Nov 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Add option to show full context by default, in file-per-page

Project Member Reported by nparker@chromium.org, Nov 1 2016

Issue description

When reviewing code that isn't super familiar, it's crucial to be able to show the edits within the context of the entire file.  This is really hard in the new Polymer UI since I need to click "context" in all the appropriate places. Then to move to the next file I have to close the contexts and click them all on the subsequent file.

It's also hard to scan the file when it is opened in-line with the list of files -- there should be a way to show a file-per-page -- this makes it possible to search the page for things without extraneous hits.

Thanks.
 
Status: Available (was: Untriaged)
FYI we plan on moving away from Rietveld very soon (within the next 3-4 months). See https://polygerrit.appspot.com for more info.
Status: WontFix (was: Available)
Ah never mind then.  polygerrit appears to allow a full-page + full-context diff setting.

Sign in to add a comment