Merge lp://staging/~blr/loggerhead/bug-1436483-sbs-diff-css-fix into lp://staging/loggerhead

Proposed by Kit Randel
Status: Merged
Merged at revision: 489
Proposed branch: lp://staging/~blr/loggerhead/bug-1436483-sbs-diff-css-fix
Merge into: lp://staging/loggerhead
Diff against target: 12 lines (+1/-1)
1 file modified
loggerhead/static/css/diff.css (+1/-1)
To merge this branch: bzr merge lp://staging/~blr/loggerhead/bug-1436483-sbs-diff-css-fix
Reviewer Review Type Date Requested Status
Riccardo Padovani (community) Approve
Loggerhead Reviewers Pending
Review via email: mp+263425@code.staging.launchpad.net

Commit message

Fix wrapping layout of sbs diff.

Description of the change

A fix to correct the rending of line numbers in diffs (https://bugs.launchpad.net/loggerhead/+bug/310255) unfortunately affected the rendering of sbs diffs (https://bugs.launchpad.net/loggerhead/+bug/1436483).

This µbranch fix should satisfy both.

To post a comment you must log in.
Revision history for this message
Riccardo Padovani (rpadovani) wrote :

I tested on Chromium and indeed it works well.

There is a regression on bug #310255 if the width of the browser is < ~1050px, but I think could be acceptable (or at least it improves a lot the actual situation)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches