Consolidate and fix the expand diff links.

Review Request #3398 — Created Oct. 4, 2012 and submitted

Information

Review Board
release-1.6.x

Reviewers

Consolidate and fix the expand diff links.

There's now a couple template tags for rendering the diff links in a
simpler way. This ends up going through a single utility template to
render each link.

As part of this, we now have a smarter place to handle the diff headers.
We no longer show expansion links next to headers showing
functions/classes that are not contained within that collapsed region.
The headers are still shown, but you can't expand.
Tested all the expand and collapse links.

Checked all headers across various diffs to make sure that the only expandable
ones were those that contained the function/class within the collapsed region.
chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Loading...