Opened 8 years ago

Closed 8 years ago

#4714 closed Bug (fixed)

JavaScript error when we edit Flash Property

Reported by: ohtani Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: UI : Dialogs Version: SVN (CKEditor) - OLD
Keywords: Confirmed Review+ Cc:

Description

When we edit Flash property with Internet Explore, we get JavaScript error.

  1. Insert Flash into CKEditor.
  2. Open Flash Property Editor.
  3. Submit it. We got error.

The attached is the patch to fix this propblem.

Attachments (2)

plugin_js.diff (702 bytes) - added by ohtani 8 years ago.
patch for _source/plugins/selection/plugin.js
4714.patch (637 bytes) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (8)

Changed 8 years ago by ohtani

Attachment: plugin_js.diff added

patch for _source/plugins/selection/plugin.js

comment:1 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: WorksForMe added

I'm trying to reproduce this issue in both the online demo and the nightly build and I have no issues. I'm using IE8 in standard and compatibility modes.

Do you have more information on how to reproduce it?

comment:2 Changed 8 years ago by Garry Yao

Keywords: Confirmed added; WorksForMe removed
Milestone: CKEditor 3.1
Owner: set to Garry Yao
Status: newassigned

Confirmed with IE6.

comment:3 Changed 8 years ago by ohtani

I am using CKEditor 3.0.1 with IE6, IE7 and IE8.

I tried in the online demo 2weeks ago, and reproduced this issue.

Changed 8 years ago by Garry Yao

Attachment: 4714.patch added

comment:4 Changed 8 years ago by Garry Yao

Component: GeneralUI : Dialogs
Keywords: Review? added
Version: SVN (CKEditor)

comment:5 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:6 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4573].

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