Closed
Bug 411792
Opened 18 years ago
Closed 18 years ago
white-space:-moz-pre-wrap wraps newlines incorrectly
Categories
(Core :: Layout: Text and Fonts, defect, P3)
Tracking
()
RESOLVED
FIXED
People
(Reporter: roc, Assigned: roc)
References
Details
(Keywords: regression, testcase)
Attachments
(2 files, 1 obsolete file)
|
105 bytes,
text/html
|
Details | |
|
4.22 KB,
patch
|
smontagu
:
review+
|
Details | Diff | Splinter Review |
A line containing a single pre-wrap word followed by a newline breaks before the newline, which means the newline ends up on the next line by itself.
|
Assignee | |
Comment 1•18 years ago
|
||
The fix is very simple, we just detect when we're about to break before a preformatted newline and break after it instead.
I considered modifying linebreaker code to disallow breaks before a newline character but that ended up being more complicated.
Attachment #296441 -
Flags: review?(smontagu)
|
|
Assignee | |
Comment 2•18 years ago
|
||
This is a regression from Gecko 1.8.1 and should block.
Flags: blocking1.9?
Keywords: regression,
testcase
|
|
Assignee | |
Comment 3•18 years ago
|
||
Sorry, that was the wrong patch.
Attachment #296441 -
Attachment is obsolete: true
Attachment #296443 -
Flags: review?(smontagu)
Attachment #296441 -
Flags: review?(smontagu)
|
|
Assignee | |
Updated•18 years ago
|
Whiteboard: [needs review]
|
|
Assignee | |
Updated•18 years ago
|
Flags: blocking1.9? → blocking1.9+
Priority: -- → P3
|
||
Updated•18 years ago
|
Attachment #296443 -
Flags: review?(smontagu) → review+
|
|
Assignee | |
Comment 4•18 years ago
|
||
checked in with test
Status: NEW → RESOLVED
Closed: 18 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [needs review]
You need to log in
before you can comment on or make changes to this bug.
Description
•