Merge lp://staging/~cjwatson/loggerhead/view-fix-line-height into lp://staging/loggerhead
Proposed by
Colin Watson
| Status: | Merged |
|---|---|
| Merged at revision: | 493 |
| Proposed branch: | lp://staging/~cjwatson/loggerhead/view-fix-line-height |
| Merge into: | lp://staging/loggerhead |
| Diff against target: |
25 lines (+4/-0) 2 files modified
NEWS (+3/-0) loggerhead/static/css/view.css (+1/-0) |
| To merge this branch: | bzr merge lp://staging/~cjwatson/loggerhead/view-fix-line-height |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Tom Wardill (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Set line-height on <pre> elements in file views.
Description of the change
This fixes misaligned line numbers in some browsers, e.g. Firefox; I think the browser was picking a different default line-height for empty vs. non-empty lines.
To post a comment you must log in.
Revision history for this message
| Tom Wardill (twom) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
