Ticket #7904 (closed Bug: fixed)

Opened 4 years ago

Last modified 4 years ago

IE: We can't apply text/background color with out focus in editor body.

Reported by: satya Owned by: Saare
Priority: Normal Milestone: CKEditor 3.6.1
Component: General Version: 3.6.1
Keywords: IBM Cc: damo, jamescun, tmonahan

Description

To reproduce the defect:

Open Ajax sample, with out focus in Editor body click on text or background color drop down menu and select a color.

Expected Result: cursor goes to editor body and the selected text/background color applies to the empty paragraph.

Actual Result: we are getting an error and the error is "null is null or not an object". Cursor goes out of editor body and when we keep cursor in editor body using mouse and type the text the selected text/background color is not applied to the typed text.

Tested in IE6, IE7, IE8 & IE9

Attachments

7904.patch (1.6 KB) - added by Saare 4 years ago.
7904_2.patch (674 bytes) - added by garry.yao 4 years ago.
7904_3.patch (1.8 KB) - added by garry.yao 4 years ago.
7904_4.patch (2.2 KB) - added by Saare 4 years ago.

Change History

comment:1 Changed 4 years ago by j.swiderski

  • Status changed from new to confirmed

The JS error:
Message: 'null' is empty or not an object
Line: 1514
Char: 4
URI: /_source/plugins/styles/plugin.js

Just like in #7905 I've been able to confirm it only for IE8 and IE9 from rev [6920]

comment:2 Changed 4 years ago by fredck

  • Milestone set to CKEditor 3.6.1

It looks like we need to review something after [6920].

Changed 4 years ago by Saare

comment:3 Changed 4 years ago by Saare

  • Owner set to Saare
  • Status changed from confirmed to review

Patch fixes also #7905 and #7919.

comment:4 Changed 4 years ago by garry.yao

  • Status changed from review to review_failed

With the patch if previous focus is from another editor instance, focus is absorbed into the current one, which will break things like shared toolbar.

Changed 4 years ago by garry.yao

comment:5 Changed 4 years ago by garry.yao

Other browsers are free of this trouble as the confirm click blurs the float panel properly.

comment:6 Changed 4 years ago by Saare

  • Status changed from review_failed to review

Changed 4 years ago by garry.yao

comment:7 Changed 4 years ago by garry.yao

Have to update the patch due to #7916, which is quite hard to get fixed with the same way in second patch.

comment:8 Changed 4 years ago by Saare

  • Status changed from review to review_failed

With 7904_3 this ticket's TC fails while #7905, #7919 and #7916 are resolved.

Changed 4 years ago by Saare

comment:9 Changed 4 years ago by Saare

  • Status changed from review_failed to review

comment:10 Changed 4 years ago by garry.yao

  • Status changed from review to review_passed

comment:11 Changed 4 years ago by Saare

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

Fixed with [6985].

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