Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#4696 closed Bug (wontfix)

Insert Page break not splitting table and list

Reported by: Garry Yao Owned by:
Priority: Normal Milestone:
Component: General Version: SVN (CKEditor) - OLD
Keywords: HasPatch Cc:

Description

Reproducing Procedures

  1. Open a sample page, load the editor with the following content with selection:
    <ol>
    	<li>item1</li>
        <li>^item2</li>
    </ol>
    
  2. Click on 'Insert Page Break...' button.
  3. Switch to 'Source' mode.
  • Actual Result:
    <div style="page-break-after: always;">
    	<span style="display: none;">&nbsp;</span></div>
    <ol>
    	<li>
    		item1</li>
    	<li>
    		item2</li>
    </ol>
    
  • Expected Result:
    <ol>
    	<li>
    		item1</li>
    </ol>
    <div style="page-break-after: always;">
    	<span style="display: none;">&nbsp;</span></div>
    <ol>
    	<li>
    		item2</li>
    </ol>
    

Attachments (1)

4696.patch (4.9 KB) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 10 years ago by Garry Yao

Keywords: HasPatch added

I fired this ticket when I was writing the drupal pagebreak plugin, where I found currently we're missing a solution for complete content splitting ( split up everything to reach body ).

Ticket Test added at :
http://ckeditor.t/tt/4614/1.html.

Changed 10 years ago by Garry Yao

Attachment: 4696.patch added

comment:2 Changed 10 years ago by Garry Yao

Milestone: CKEditor 3.1CKEditor 3.2

comment:3 Changed 10 years ago by Frederico Caldeira Knabben

Resolution: wontfix
Status: newclosed

Page break is a printing feature. It has no impact in the visualization of the page in the browser. If the page break is inside a table or a list, it will bring no issues and the page break will still work properly when printing.

By breaking lists or table instead, we break the visual representation of the content into the browser, which is a bad thing.

I know this feature is inspired in the Drupal breaks plugin, but at the end we found out that breaking the page structure is not desired by the users, and so we've provided a different solution for it there.

comment:4 Changed 10 years ago by Frederico Caldeira Knabben

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