Fixed the alignment of line numbers on moved rows in diffs.

Review Request #5551 — Created Feb. 26, 2014 and submitted

Information

Review Board
release-2.0.x
def89e1...

Reviewers

A couple fixes were attempted in the past to fix alignment of line
numbers whne moved rows came into play. The reason they didn't work was
the (fairly reasonable) assumption that every moved row had the
.moved-row CSS class. This was only set for the rows with move flags,
though.

Since that's a pretty decent assumption to make, let's make it true.
Every moved row now has the .moved-row class applied.

Viewed a diff with a bunch of move information. Before this change,
the line numbers were noticeably unaligned. After this change, they
all appear aligned.

david
  1. Ship It!

  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Loading...