Fix review request field sizing issues and hide unused fields.

Review Request #3541 — Created Nov. 20, 2012 and submitted

Information

Review Board
master

Reviewers

Fix review request field sizing issues and hide unused fields.

Review request fields were never shown very well. They had a hacky setup
that caused them to jump when the content changed, such as more text or
an inline editor being shown. This was ugly and I just won't stand it
anymore.

Now instead of a 4 column approach (which was really the core issue, as
it meant that the table was reevaluating its layout based on any new
content all the time and causing things to jump), we have two tables
floated next to each other, each taking up 50% of the space.

These tables will no longer intrude on each others' space, preventing
things from jumping around.

A nice side-benefit is that when resized really small (such as on mobile
devices), the tables will themselves wrap, moving to a single column.

Along with this, I've changed the template to not show the Change Number
field unless one is present, and not show Repository unless it's a
repository-backed review request. This simplifies things in certain
cases.
Tested on Chrome and Firefox. I tried resizing, opening and closing editors.
Nothing shifted weirdly anymore. It almost looked polished.
Description From Last Updated

Can we move these to defs.less?

daviddavid
david
  1. Looks pretty good.
  2. reviewboard/static/rb/css/reviews.less (Diff revision 1)
     
     
     
    Can we move these to defs.less?
  3. 
      
chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to master (c663d81)
Loading...