Ticket #7581 (closed Bug: fixed)

Opened 3 years ago

Last modified 3 months ago

IE Crashes when we try to insert a new list item in the pasted list.

Reported by: satya Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.6.1
Component: Plugin : Paste from Word Version: 3.5.3
Keywords: IBM IE Cc: monahant@…, jamcunni@…

Description

To reproduce the defect:

  1. Copy the list from any of the attached word doc's and paste it using Paste from Word dialog.
  1. See that the list is pasted properly.
  1. Keep cursor next to one of the list items and press Enter.

Expected Result:

New list items starts at the same level.

Actual Result:

Browser crashes . Tested against IE6,IE7

Attachments

Numbering.docx (14.2 KB) - added by satya 3 years ago.
stupidList.docx (11.0 KB) - added by satya 3 years ago.
7581.patch (1.0 KB) - added by garry.yao 3 years ago.

Change History

Changed 3 years ago by satya

Changed 3 years ago by satya

comment:1 Changed 3 years ago by tmonahan

  • Cc monahant@… added

comment:2 Changed 3 years ago by jamescun

  • Cc jamcunni@… added

comment:3 Changed 3 years ago by j.swiderski

  • Status changed from new to pending

Works for me. I could not reproduce it.

Does it happen always with any element or some kind of special condition has to be met?

comment:4 Changed 3 years ago by j.swiderski

  • Status changed from pending to confirmed

I have managed to confirm it. This has been true for IE6 and IE7 from CKEditor version 3.5.3 revision [6616]

  1. Open ajax sample
  2. Copy "stupidList" with CRTL+A CRTL+C
  3. Paste it using paste from word dailog. (If IE asks you whether to allow... press allow)
  4. Place the cursor behind "n" and press enter, the browser will hang.

comment:5 Changed 3 years ago by fredck

  • Version changed from 3.5.3 to 3.5.4 (SVN - trunk)

comment:6 Changed 3 years ago by wwalc

  • Milestone set to CKEditor 3.5.4

comment:7 Changed 3 years ago by garry.yao

  • Keywords IE added
  • Component changed from General to Core : Lists

Confirmed on old IEs(6&7), probably because of the depth of nested list has brought too much a burden for the browser, I would be hard to have it in this milestone though and it's not a trunk regression.

comment:8 Changed 3 years ago by fredck

  • Version changed from 3.5.4 (SVN - trunk) to 3.5.3
  • Milestone changed from CKEditor 3.5.4 to CKEditor 3.6.1

Ops, I made a mistake with the version. I wanted to change the milestone instead.

Anyway, we need to go out with the 3.5.4 now, so I'm postponing it, to be dealt right after the release.

Changed 3 years ago by garry.yao

comment:9 Changed 3 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from confirmed to review

comment:10 Changed 3 years ago by fredck

  • Status changed from review to review_passed

comment:11 Changed 3 years ago by garry.yao

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

Fixed with [6950].

comment:12 Changed 3 months ago by fredck

  • Component changed from Core : Lists to Plugin : Paste from Word
Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy