Merge lp:~ya-bo-ng/ubuntu-manpage-repository/1783869-fix-pre-styling into lp:ubuntu-manpage-repository
Proposed by
Anthony Dillon
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | David Britton | ||||
Approved revision: | 224 | ||||
Merged at revision: | 224 | ||||
Proposed branch: | lp:~ya-bo-ng/ubuntu-manpage-repository/1783869-fix-pre-styling | ||||
Merge into: | lp:ubuntu-manpage-repository | ||||
Diff against target: |
12 lines (+2/-0) 1 file modified
www/assets/light/css/styles.css (+2/-0) |
||||
To merge this branch: | bzr merge lp:~ya-bo-ng/ubuntu-manpage-repository/1783869-fix-pre-styling | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
David Britton (community) | Approve | ||
Review via email: mp+351432@code.launchpad.net |
Commit message
LP: #1783869 - Fix the pre formatting
Description of the change
Add pre formatting to pre elements and made them scrollable when the content does not fit.
To post a comment you must log in.
Hi Ant, I think it has improved the situation, but I think there is still a couple things that would be nice to see.
Could we give the whole page a minimum fixed width, and not each individual section? If perhaps we put <pre> tags around the whole thing?
e.g., http:// man7.org/ linux/man- pages/man1/ ac.1.html
I took this screenshot showing the jaggedness of having each section broken up
https:/ /i.imgur. com/CB4X61f. png
Also, each section has a border even if it's not scrollable, which looks a bit wonky.