Opened 8 years ago

Closed 5 years ago

#4079 closed Bug (expired)

[FF] Create list error no multiple items

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone:
Component: Core : Lists Version:
Keywords: Firefox Cc:

Description (last modified by Garry Yao)

Reproducing Procedures

  1. Open the 'replace by class' sample page in Firefox;
  2. Load the document with the following content and selection with CTRL-A:
    [<p>line1</p>
    <p>line2</p>]
    
  3. Click on 'Insert Bulleted List' button;
    • Actual Result: Javascript error thrown.

Attachments (1)

4079.patch (2.4 KB) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 8 years ago by Garry Yao

Description: modified (diff)
Keywords: Firefox added; FF removed

Refine the description.

comment:2 Changed 8 years ago by Garry Yao

Keywords: Confirmed added
Owner: set to Garry Yao
Status: newassigned

Regression of [3930] where the function parameter refactoring is not complete.

Changed 8 years ago by Garry Yao

Attachment: 4079.patch added

comment:3 Changed 8 years ago by Garry Yao

Keywords: Review? added

Note that the incorrect result of the list is out the scope of this ticket.

comment:4 in reply to:  2 Changed 8 years ago by Frederico Caldeira Knabben

Replying to garry.yao:

Regression of [3930] where the function parameter refactoring is not complete.

Regression means it was working before [3930]... so there is no regression here.

comment:5 Changed 8 years ago by Garry Yao

Reproducing Procedures

  1. Open the 'replace by class' sample page in Firefox;
  2. Clear all content with 'Ctrl-A' and then 'Del';
  3. Type "line1" + ENTER;
  4. Type "line2";
  5. Select all with 'Ctrl-A' and click on 'Insert/Remove Bulleted List' button;
    • Actual Result: Javascript error thrown.

comment:6 Changed 8 years ago by Garry Yao

Above is a more precise way to catch it on Firefox3.0.12.

comment:7 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: WorksForMe added; Confirmed Review? removed
Milestone: CKEditor 3.0CKEditor 3.x
Priority: HighNormal

Me and Tobiasz are not able to reproduce this issue. Let's wait for others to confirm it before proceeding on changes here.

comment:8 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.x

Milestone CKEditor 3.x deleted

comment:9 Changed 5 years ago by Jakub Ś

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