Am 27.01.2017 um 07:07 schrieb David Ostrovsky: > It would be also cool, if you could look into new and shiny gerrit UI, > based on JavaScript Polymer framework, and provide feedback. You can > see the new UI in action upstream by switching in footer to new UI. > Only change screen is implemented for now. All diffs on one page is > already implemented: > > https://gerrit-review.googlesource.com/
Generally I like the new UI - if just for the - per file expandable - inline diff :-) But switching the "info" and "commit message" frames produces a lot of unused / white area on the right, and uses more precious vertical space. Especially since the left "info" frame doesn't expand much horizontally, so can't / won't display multiple reviewers in one row, but instead the commit message expands, which almost ever has a fixed maximum width. I first had a look at https://gerrit-review.googlesource.com/c/95455/?polygerrit=1 but it's probably event better to see, if you check https://gerrit-review.googlesource.com/c/94813/?polygerrit=1 Now there is some of the white space used by the relation chain. Just decrease the browser horizontally. To save some vertical space it would also help to merge Reviewers, Code-Review and Verified into a table, so this won't duplicate name lists, like: Reviewer | Verified | Code Me | +1 | +1 You | | -1 _______________________________________________ LibreOffice mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/libreoffice
