← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] ~lgp171188/launchpad:diff-viewer-fix-text-overflow into launchpad:master

 

We already have text wrapping there, which causes the long lines to wrap without using horizontal scrollbars.

Be default, it breaks the lines at word delimiters (whitespace, dash, etc.). By having `overflow-wrap: break-word;` we allow wrapping by breaking longs words, which is the case in your example.

You can find some information on disadvantages of the horizontal scrollbars here: https://www.nngroup.com/articles/horizontal-scrolling/

In general, they offer bad user experience.
-- 
https://code.launchpad.net/~lgp171188/launchpad/+git/launchpad/+merge/424302
Your team Launchpad code reviewers is requested to review the proposed merge of ~lgp171188/launchpad:diff-viewer-fix-text-overflow into launchpad:master.



References