Ticket #8673 (closed Bug: fixed)

Opened 2 years ago

Last modified 2 months ago

Chrome: cannot select and remove Page Break (pagebreak)

Reported by: tomek Owned by: m.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

comment:1 Changed 2 years ago by j.swiderski

  • Status changed from new to confirmed
  • Keywords Webkit Opera added; chrome pagebreak selection delete remove removed
  • Version changed from 3.6.3 (SVN - trunk) to 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 4 months ago by davidw

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 2 months ago by m.lewandowski

  • Owner set to m.lewandowski
  • Status changed from confirmed to assigned

comment:4 Changed 2 months ago by m.lewandowski

  • Status changed from assigned to review

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

comment:5 Changed 2 months ago by m.lewandowski

  • Status changed from review to review_passed

comment:6 Changed 2 months ago by m.lewandowski

  • Milestone set to CKEditor 4.3.3

comment:7 Changed 2 months ago by m.lewandowski

  • Status changed from review_passed to closed
  • Resolution set to fixed

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

Last edited 2 months ago by m.lewandowski (previous) (diff)

comment:8 Changed 2 months ago by m.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 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy