237275 - Comb wrapping for README and license; shouldn't have newlines

Status: RESOLVED FIXED

(SeaMonkey :: Installer, defect)

Description

[Akkana Peck]

The new installer stages which show the README and MPL show comb wrapping (every
other line is only a few characters long) using the default Debian fonts (this
will also be true for anyone who uses large fonts for accessibility reasons). 
The textarea isn't wide enough to show the lines at their actual length.  

This is cosmetic only, but gives a bad impression of mozilla since it's the
first thing the user sees.

The easy fix: remove the newlines within paragraphs in the installer's copy of
these documents, and let the textarea word-wrap the lines according to the
actual font in use.

Comment 1

[Andrew Schultz]

Attachment: patch

this
1) makes the Welcom dialog (README) wrap on words.
2) removes newlines from README and removes most of the indenting, which looks
bad on only the first line.

the README content is not changed even though it's wrong (bug 58859)

Comment 2

[Andrew Schultz]

*** Bug 127433 has been marked as a duplicate of this bug. ***

Comment 3

[Andrew Schultz]

Comment on attachment 143773 [details] [diff] [review]
patch

dveditz: the gtk portion of this patch got r= and your sr= in bug 127433.

Comment 4

[Daniel Veditz [:dveditz]]

Comment on attachment 143773 [details] [diff] [review]
patch

r/sr=dveditz

Comment 5

[Andrew Schultz]

Comment on attachment 143773 [details] [diff] [review]
patch

this patch aesthetically improves the README by properly wrapping words.  very
low risk

Comment 6

[Andrew Schultz]

see also bug 237342.
I'm not sure removing the newlines from the license is the best thing to do
since that would also removing the indenting, which is helpful, at least
compared to the README, where it doesn't really do much.

Comment 7

[chris hofmann]

Comment on attachment 143773 [details] [diff] [review]
patch

a=chofmann for 1.7b

Comment 8

[Christian :Biesinger (don't email me, ping me on IRC)]

checked in