Opened 8 years ago

Last modified 7 years ago

#8281 confirmed Bug

Internet Explorer 7.x crashes when changing an item from a bulleted list into numbered list

Reported by: wittmakl Owned by:
Priority: Normal Milestone:
Component: Core : Lists Version: 3.5
Keywords: IE7 Cc:

Description

Steps to recreate:

  • Make a numbered list
  • highlight one or several lines within the numbered list (make sure that the first or last line of the list is not highlighted)
  • try to change these lines from nubered list to 'bulleted' list
  • Internet Explorer will crash

(works vice versa too)

This issue only occurs until IE7.x it does not occur with other browsers or IE version higer than 8.0 For our company it is vital to use CKEdit with IE7.x as our clinical information system is not validated for IE versions higher than IE7.x

Attachments (1)

CKEdit_List_Bug.zip (269.5 KB) - added by wittmakl 8 years ago.

Download all attachments as: .zip

Change History (5)

Changed 8 years ago by wittmakl

Attachment: CKEdit_List_Bug.zip added

comment:1 Changed 8 years ago by Jakub Ś

Component: GeneralCore : Lists
Keywords: list removed
Status: newconfirmed
Version: 3.6.13.5

Reproducible from CKE 3.5 in IE7 only.

I have tested the issue with vice versa scenario. I have created 5 bulleted items, selected three of them from item2 to item4 and pressed numbered list button.

comment:2 Changed 8 years ago by Audrey Tang

This is still present in IE7 for CKEditor nightly; switching from OL=>UL and UL=>OL crashes the browser.

Would it be possible to disable the corresponding commands for IE7 when we know we're in a list that'd cause this kind of crash?

comment:3 Changed 8 years ago by Audrey Tang

I've applied a naïve patch locally that simply disallows list-changing operation for IE6/7:

// list/plugin.js
        function changeListType( editor, groupObj, database, listsCreated )

            if (CKEDITOR.env.ie6Compat || CKEDITOR.env.ie7Compat) {
                alert("Sorry, changing list types is unsupported in IE7 and earlier versions.");
                return;
            }
// ...

comment:4 Changed 7 years ago by Jakub Ś

Keywords: IE7 added

#9035 was marked as duplicate.

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