Project

General

Profile

Actions

Defect #866

closed

Unusable side-by-side diff - horizontal scrolling

Added by Nils Adermann about 16 years ago. Updated about 16 years ago.

Status:
Closed
Priority:
High
Assignee:
-
Category:
UI
Target version:
Start date:
2008-03-15
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

The side by side diff is pretty much unusable because the code is neither wrapped when it's too long nor is the horizontal scrollbar anywhere reachable (there is one at the very bottom of the page for the div). You should either be able to scroll the entire page horizontally. That way you don't need to scroll all the way down to use the horizontal scrollbar but can scroll horizontally anywhere on the page using your browser. Or lines should be wrapped to not let this happen at all. For an example see http://code.phpbb.com/repositories/diff/5/trunk/phpBB/includes/functions_compress.php?rev_to=5853&rev=6015&action=diff&id=5&controller=repositories&path=trunk%2FphpBB%2Fincludes%2Ffunctions_compress.php&type=sbs


Related issues

Is duplicate of Redmine - Defect #200: scm view diffsClosed

Actions
Actions #1

Updated by Nils Adermann about 16 years ago

Well the demo link doesn't show it anymore as I removed class="autoscroll" from the div as I'd like to be able to use it ;-)

Actions #2

Updated by Jean-Philippe Lang about 16 years ago

  • Status changed from New to Resolved
  • Target version set to 0.7

It was fixed in r1020. Diffs are now wrapped:
http://www.redmine.org/repositories/diff/redmine?rev=1249&type=sbs

I think you can fix it manually by adding this to your css:

table.filecontent td.line-code pre {
    white-space: pre-wrap; /* CSS2.1 compliant */
    white-space: -moz-pre-wrap; /* Mozilla-based browsers */
    white-space: -o-pre-wrap; /* Opera 7+ */
}
Actions #3

Updated by Nils Adermann about 16 years ago

Thanks, that works.

Actions #4

Updated by Jean-Philippe Lang about 16 years ago

  • Status changed from Resolved to Closed
  • Resolution set to Fixed
Actions

Also available in: Atom PDF