Closed
Bug 277035
Opened 20 years ago
Closed 20 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•20 years ago
|
||
|
|
Reporter | |
Comment 2•20 years ago
|
||
(In reply to comment #0) > ...Both MSIE Win and Safari > generate only two pages. That is they generate ONE page.
|
Assignee | |
Updated•20 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•20 years ago
|
||
|
|
Assignee | |
Updated•20 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•20 years ago
|
||
Rearranged the expression according to comment 4... Checked in 2005-01-16 07:30 PDT -> FIXED
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•