Opened 13 years ago

Closed 11 years ago

Last modified 11 years ago

#8673 closed Bug (fixed)

Chrome: cannot select and remove Page Break (pagebreak)

Reported by: tomek Owned by: Marek Lewandowski
Priority: Normal Milestone: CKEditor 4.3.3
Component: Core : Selection Version: 3.6
Keywords: Webkit Opera Cc:

Description

  1. Use a Chrome browser (I have 17.0.963.26 (Developer Build 0 Linux))
  2. Go to http://ckeditor.com/demo
  3. Insert the Page Break and select it (click it)
  4. Press a 'Delete' key

The status bar shows "body pagebreak", however pressing a 'Delete' key does nothing.

Change History (8)

comment:1 Changed 13 years ago by Jakub Ś

Keywords: Webkit Opera added; chrome pagebreak selection delete remove removed
Status: newconfirmed
Version: 3.6.3 (SVN - trunk)3.6

The issue has been reproducible on all operating systems on Webkit and Opera browsers from CKEditor 3.6 rev [6904]

It is not possible to select page break and delete it using either Delete or Backspace key.

NOTES:

  • In Windows pressing backspace acts like back button in this case.
  • It is still possible to select page break and delete it. Just place the cursor below page break and press Shift+left arrow few times to select the page break. Pressing Backspace or delete will work in this case.

comment:2 Changed 11 years ago by David N. Welton

I worked around this by adding the following to my own code, snipped from the pagebreak plugin itself:

    if (CKEDITOR.env.chrome) {
	    editor.document.on( 'click', function( evt ) {
	        var target = evt.data.getTarget();
	        if ( target.is( 'div' ) && target.hasClass( 'cke_pagebreak' ) ) {
		    editor.getSelection().selectElement( target );
                }
	    } );
        }

comment:3 Changed 11 years ago by Marek Lewandowski

Owner: set to Marek Lewandowski
Status: confirmedassigned

comment:4 Changed 11 years ago by Marek Lewandowski

Status: assignedreview

With code from PR allows Chrome to select page break. Pushed to t/8673 at dev.

comment:5 Changed 11 years ago by Marek Lewandowski

Status: reviewreview_passed

comment:6 Changed 11 years ago by Marek Lewandowski

Milestone: CKEditor 4.3.3

comment:7 Changed 11 years ago by Marek Lewandowski

Resolution: fixed
Status: review_passedclosed

Fixed with git:1eb5d94f61f1f9 (merged to master) at dev.

Last edited 11 years ago by Marek Lewandowski (previous) (diff)

comment:8 Changed 11 years ago by Marek Lewandowski

With revision git:a335dadb7be2 we included all webkit based browsers, coz it turned out that Safari had this issue too.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy