Revamp the review request layout.

Review Request #4919 — Created Nov. 8, 2013 and submitted

Information

Review Board
master

Reviewers

Revamp the review request layout.

We've had the same look to the review request box since the dawn of
time. With 2.0, we're changing that to take up less vertical space,
to bring a little more consistency with some of the fields, and to
highlight the main content of the review request.

The main information (Summary, Description and Testing Done) are now
shown one after another, with the same font style used for the labels.

On the right-hand side is the list of fields we previously had between
Summary and Description. They're a single column now. This takes the
space that was often just left unused within the Description/Testing
Done fields (since those fields were often far wider than the contents).

Simplifying this into one column will also make it easier to lay out
fields provided by hooks.

The Screenshots, File Attachments and Issue Summary are displayed after
all the fields, as before. Their labels match the style of the Summary,
Description and Testing Done labels.

The Issue Summary table has been cleaned up a bit to have the filters
and the counters on the same row, reducing the amount of vertical space.

Played around with this in Chrome and in Firefox. It made much better
use of space, and didn't hide Testing Done when I had longer descriptions.

Made sure that the field contents could wrap and that that wouldn't
interfere with the inlineEditor (once the accompanying Djblets bug was
fixed).


Description From Last Updated

The pencils being right-aligned for the multiline fields and left-aligned for everything else is making me twitch. Can we move …

daviddavid

Can we move the nested items over to the right just a smidge more? It looks like there's no indentation …

daviddavid

Can we define a variable for this?

daviddavid
david
  1. I haven't looked at the code yet.

  2. The pencils being right-aligned for the multiline fields and left-aligned for everything else is making me twitch. Can we move them over next to the "Description" and "Testing Done" labels?

  3. Can we move the nested items over to the right just a smidge more? It looks like there's no indentation at all.

  4. 
      
chipx86
david
  1. Looks like a pretty good start. Probably there will be more tweaking after we use it for a bit.

  2. reviewboard/static/rb/css/reviews.less (Diff revision 2)
     
     

    Can we define a variable for this?

  3. 
      
chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Loading...