Page MenuHomePhabricator

Files that are completely new or have only additive/subtractive change are harder to read in code reviews with split view
Closed, InvalidPublic

Description

When reviewing code, if a file is brand new or has either only additions or only deletions the files are actually harder to read because the default split view wastes a lot of empty space to the left or right of the file. It would be nice if the view recognized this and showed these files in the unified view automatically. I know you can choose to render the file in unified view but that is an extra step someone has to take. This seems like an easy change which should improve the code review process immensely since this type file change does seem to happen a lot.

Event Timeline

chad closed this task as Invalid.Dec 20 2016, 3:13 AM
chad added a subscriber: chad.

Sorry, we don't take UI suggestions as feature request tasks, but do value the input and will keep it in mind.

@chad Ah bummer. Is there an official channel you guys recommend for feedback of this type?

chad added a comment.Dec 20 2016, 3:18 AM

We take design feedback during product launches, and no other time.