• 
      

    Take the review banner into consideration in diff anchor offsets.

    Review Request #6912 — Created Feb. 6, 2015 and submitted — Latest diff uploaded

    Information

    Review Board
    release-2.0.x
    f6068ba...

    Reviewers

    A recent change reduced the offset from the top of the page to the
    selected anchor in the diff viewer, in order to show more of the diff.
    This ended up causing the code to be covered up when showing the review
    draft banner.
    
    We now factor in the banner's height when choosing the offset from the
    top of the page.

    Scrolled through the anchors with the keyboard shortcuts. Saw the small
    gap between the top of the page and the anchor.

    Added a comment and saw the draft banner appear. Scrolled through some
    more, and saw the offset took the banner into consideration.