Merge lp:~cjwatson/loggerhead/view-fix-line-height into lp:loggerhead

Proposed by Colin Watson on 2019-06-20
Status: Merged
Merged at revision: 493
Proposed branch: lp:~cjwatson/loggerhead/view-fix-line-height
Merge into: lp: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:~cjwatson/loggerhead/view-fix-line-height
Reviewer Review Type Date Requested Status
Tom Wardill 2019-06-20 Approve on 2019-06-20
Review via email: mp+369102@code.launchpad.net

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.
Tom Wardill (twom) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'NEWS'
2--- NEWS 2015-03-17 11:38:32 +0000
3+++ NEWS 2019-06-20 16:20:26 +0000
4@@ -27,6 +27,9 @@
5 - Add some more minor UI changes to do with rounded corners. (Cruz
6 Bishop)
7
8+ - Set line-height on <pre> elements in file views, fixing misaligned
9+ line numbers in some browsers. (Colin Watson)
10+
11 1.18.2 [12Sep2012]
12 ------------------
13
14
15=== modified file 'loggerhead/static/css/view.css'
16--- loggerhead/static/css/view.css 2012-03-19 04:37:07 +0000
17+++ loggerhead/static/css/view.css 2019-06-20 16:20:26 +0000
18@@ -28,6 +28,7 @@
19 .viewLine pre,
20 .viewCont pre {
21 font-family: 'UbuntuBeta Mono', 'Ubuntu Mono', monospace;
22+ line-height: 1.1;
23 }
24 .viewLine pre {
25 text-align: right;

Subscribers

People subscribed via source and target branches