• 
      

    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:
    Completed