Closed
Bug 277035
Opened 21 years ago
Closed 21 years ago
[FIX] page break is not removed by display:none
Categories
(Core :: Layout, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mikecaines, Assigned: MatsPalmgren_bugz)
Details
(Keywords: testcase)
Attachments
(2 files)
|
138 bytes,
text/html
|
Details | |
|
2.04 KB,
patch
|
dbaron
:
review+
dbaron
:
superreview+
|
Details | Diff | Splinter Review |
See attached testcase.
The second DIV's style attribute contains "page-break-before:always" and
"display:none" but print layout still generates two pages. Exhibits same
behaviour in quirks and standards compliance modes. Both MSIE Win and Safari
generate only two pages.
|
Reporter | |
Comment 1•21 years ago
|
||
|
|
Reporter | |
Comment 2•21 years ago
|
||
(In reply to comment #0)
> ...Both MSIE Win and Safari
> generate only two pages.
That is they generate ONE page.
|
Assignee | |
Updated•21 years ago
|
Assignee: nobody → mats.palmgren
Severity: major → normal
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: testcase
OS: Windows XP → All
Summary: page break is not removed by display:none → [FIX] page break is not removed by display:none
|
|
Assignee | |
Comment 3•21 years ago
|
||
|
|
Assignee | |
Updated•21 years ago
|
Attachment #170350 -
Flags: superreview?(dbaron)
Attachment #170350 -
Flags: review?(dbaron)
Comment on attachment 170350 [details] [diff] [review]
Patch rev. 1
Get rid of the null-check of |display| and move the display-none check that
you're adding to the beginning of the expression, and r+sr=dbaron.
Attachment #170350 -
Flags: superreview?(dbaron)
Attachment #170350 -
Flags: superreview+
Attachment #170350 -
Flags: review?(dbaron)
Attachment #170350 -
Flags: review+
|
|
Assignee | |
Comment 5•21 years ago
|
||
Rearranged the expression according to comment 4...
Checked in 2005-01-16 07:30 PDT
-> FIXED
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•