Opened 7 years ago

Closed 4 years ago

#6049 closed Bug (fixed)

Page-break marker dissapears in editor when div tag modified.

Reported by: Burton Owned by:
Priority: Normal Milestone:
Component: General Version: 3.3.1
Keywords: Cc: burtonrhodes@…

Description

The page-break indicator that shows where the page breaks are inside the editor will not be shown if the div tag is modified with the editor (e.g. change font for the entire page).

To Recreate:

1) Input the folloing source to start. This will show a page-break indicator in the editor window.

<p>
	Page 1</p>
<div style="page-break-after: always">
	<span style="display: none">&nbsp;</span></div>
<p>
	Page 2</p>

2) Use editor: "Select All" and change font to "Arial" & size "8".

3) Toggle "Source" twice and the page-break indicator is gone.

4) Resulting HTML in the editor is below (which the editor doesn't interpret the "page-break" div tag correctly).

<p>
	<span style="font-size: 8px"><span style="font-family: arial, helvetica, sans-serif">Page 1</span></span></p>
<p>
	&nbsp;</p>
<div style="page-break-after: always">
	<span style="font-size: 8px"><span style="font-family: arial, helvetica, sans-serif"><span style="display: none">&nbsp;</span></span></span></div>
<p>
	&nbsp;</p>
<p>
	<span style="font-size: 8px"><span style="font-family: arial, helvetica, sans-serif">Page 2</span></span></p>

The page-break indicator should be shown even though a font attribute has been introduced to the page-break div tag.

Change History (7)

comment:1 Changed 7 years ago by Burton

Cc: burtonrhodes@… added

comment:2 Changed 7 years ago by Sa'ar Zac Elias

Resolution: invalid
Status: newclosed

Although I can confirm the bug in the demo, I can not reproduce it anymore on trunk duo to the changes in the page break markup that will be available in the next release (3.3.2).

comment:3 Changed 7 years ago by Sa'ar Zac Elias

Resolution: invalid
Status: closedreopened

As mentioned in #5079, these changes were reverted so the bug is now reproducible.

comment:4 Changed 7 years ago by Sa'ar Zac Elias

Status: reopenedconfirmed

comment:5 Changed 7 years ago by Sa'ar Zac Elias

#6472 is a DUP.

comment:6 Changed 7 years ago by Sa'ar Zac Elias

Could be covered by #6103.

comment:7 Changed 4 years ago by Piotrek Koszuliński

Resolution: fixed
Status: confirmedclosed

Not reproducible any more.

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy