Ticket #3151 (closed Bug: fixed)

Opened 5 years ago

Last modified 2 years ago

Remove list command is not working

Reported by: martinkou Owned by: martinkou
Priority: Normal Milestone: CKEditor 3.0
Component: Core : Lists Version: SVN (FCKeditor) - Retired
Keywords: Review+ Cc:

Description

To reproduce:

  1. Open replacebyclass.html sample in any supported browser.
  2. Change the paragraph into a numbered list.
  3. Press numbered list button again.
  4. The list is not removed.

Attachments

3151.patch (5.4 KB) - added by martinkou 5 years ago.
3151_2.patch (677 bytes) - added by martinkou 5 years ago.
3151_3.patch (5.0 KB) - added by garry.yao 5 years ago.

Change History

comment:1 Changed 5 years ago by martinkou

The bug is found to be caused by [3194].

There're at least two major changes which is breaking plugins:

  1. editor.config.enterMode is now changed to numeric constants. Any old code still depending on them being strings is broken.
  2. CKEDITOR.dom.node::clone() now removes element ID. Any old code that depends on the old id cloning behavior is broken.

comment:2 Changed 5 years ago by martinkou

  • Keywords Review? added

Changed 5 years ago by martinkou

comment:3 Changed 5 years ago by fredck

  • Keywords Review+ added; Review? removed

My fault here. I've wrongly assumed that that the enterKey setting is to be used by the enterkey plugin only, which is totally wrong. That's the reason why this setting is inside the core config instead of being along with the plugin.

comment:4 Changed 5 years ago by martinkou

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

Fixed with [3249].

Click here for more info about our SVN system.

comment:5 Changed 5 years ago by martinkou

#3159 is a dup of this ticket.

comment:6 Changed 5 years ago by garry.yao

  • Status changed from closed to reopened
  • Keywords Review+ removed
  • Resolution fixed deleted

The problem still existed with enterMode=br.

Changed 5 years ago by martinkou

comment:7 Changed 5 years ago by martinkou

  • Keywords Review? added

Changed 5 years ago by garry.yao

comment:8 Changed 5 years ago by garry.yao

R+ for 3151_2.patch,appending TC.

comment:9 Changed 5 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:10 Changed 5 years ago by martinkou

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

Fixed with [3313].

Click here for more info about our SVN system.

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