This is due to a very long string preventing it from wrapping, in combination 
with some CSS that prevented scrolling.

A proper fix to sanely wrap long lines is coming in 2.0.3 (this week).

Christian

-- 
Christian Hammond - christ...@beanbaginc.com
Review Board - http://www.reviewboard.org
Beanbag, Inc. - http://www.beanbaginc.com

On July 1, 2014 at 6:16:44 AM, Markus Grimm (m.gr...@adnymics.com) wrote:

Hi there,

we use Review Board in version 2.0.2 and are quite satisfied with the 
functionality, especially how the new version handles multiple diff revisions.

But in the last review, there is one file which gets not displayed correctly. 
The second column, which normally displays the new change of a file, is missing 
completely (see the picture appended to this post). It's the second review now 
where this happens and always with this file only (tests/results.txt). This is 
a file which stores the output of some module tests. We check it in to the 
repository to quickly verify in the review if a developer has added module 
tests for the feature he has written and to see if other module tests are still 
running. So it changes constantly in each repository branch and with each 
review revision. But in essence, it's just a text file which changes contents, 
just like other code files. No obvious reason to fail in displaying the new 
changes, at least not for me to see.

We use git repositories. I made this review with "rbt post", edited a few code 
lines before publishing it, checked them in (also with a modified results.txt) 
and called "rbt post -r <review-id>". So the diff hast still one revision. The 
new code lines show up correctly in the diff. I didn't check the diff of 
results.txt before calling rbt post a second time with the -r parameter, so I 
cannot tell if it's display was broken before, but I will do that in the next 
review where the bug appears.

Any suggestions about this? If there's anything you guys need to track down 
this bug further (logs etc.), please let me know.
--
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
---
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to