Issue 1377, comment flag not visible in view diffs for second line

Review Request #1189 — Created Nov. 3, 2009 and submitted — Latest diff uploaded

lonico
Review Board
1377
reviewboard
Issue 1377, comment flag not visible in view diffs.  This happens when two comments are placed on 2 consecutive lines.  I believe the cause of the problem is that startRow in findLineNumRow in reviewboard/htdocs/media/rb/js/diffviewer.js already includes the offset, so we are off by 1 when we try to locate the line for the second comment, and we cannot find it. 
I applied the patch to my local version of reviewboard, and verified the second line is displayed both in a development envirnment and on a staging server.
Loading...