Opened 11 years ago

Closed 11 years ago

#3303 closed Bug (fixed)

Remove list command is broken again

Reported by: Martin Kou Owned by: Frederico Caldeira Knabben
Priority: Must have (possibly next milestone) Milestone: CKEditor 3.0
Component: General Version: SVN (CKEditor) - OLD
Keywords: Review+ Cc:

Description

To reproduce:

  1. Open replacebyclass.html in any supported browser.
  2. Put caret on the first paragraph.
  3. Press Create/Remove Numbered List button - a list is created as expected.
  4. Press Create/Remove Numbered List button again - a JavaScript error appears.

The error seems to have to do with recent changes to DOM node custom data logic.

Attachments (1)

3303.patch (867 bytes) - added by Frederico Caldeira Knabben 11 years ago.

Download all attachments as: .zip

Change History (4)

Changed 11 years ago by Frederico Caldeira Knabben

Attachment: 3303.patch added

comment:1 Changed 11 years ago by Frederico Caldeira Knabben

Keywords: Review? added
Owner: set to Frederico Caldeira Knabben
Status: newassigned

The original implementation of the "custom data" related functions where assuming that the expando property was always there, which is not true anymore.

comment:2 Changed 11 years ago by Martin Kou

Keywords: Review+ added; Review? removed

comment:3 Changed 11 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: assignedclosed

Fixed with [3354].

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