Ticket #9060 (review Bug)

Opened 2 years ago

Last modified 2 years ago

IE: closing a dialog from a nested editor doesn't unlock the selection

Reported by: alfonsoml Owned by: alfonsoml
Priority: Normal Milestone:
Component: UI : Dialogs Version:
Keywords: Cc:

Description

This seems a bit complex to follow, but it isn't so much.

Refer to the following post for extra info: http://alfonsoml.blogspot.com.es/2010/06/nesting-dialogs-in-ckeditor.html

The process is:

  • An editor instance,
  • Launches a dialog.
  • That dialog launches a new one
  • In this dialog we create a new CKEditor
  • And now we open a dialog to create a link.

After closing the link dialog the selection isn't unlocked, and that leads to weird problems (remember, this is only for IE, other browsers work fine and that might be the reason why this problem has remained hidden for so long after I started using that setup).

Now that we know the root of the problem, the fix is quite easier.

Is it possible to review it?

Attachments

9060.patch (1.4 KB) - added by alfonsoml 2 years ago.
Proposed patch
test_9060.html (4.0 KB) - added by alfonsoml 2 years ago.
Testcase

Change History

Changed 2 years ago by alfonsoml

Proposed patch

comment:1 Changed 2 years ago by alfonsoml

  • Owner set to alfonsoml
  • Status changed from new to review

The patch first moves the changes to the zIndex to a previous check for parentDialog (no real change in behavior) and then adds an extra check so the "destruction" including the unlock of the selection happens if there is no parent dialog or if that dialog belongs to another editor instance.

comment:2 Changed 2 years ago by garry.yao

Looks like a twister, can we have a tick test for this issue?

Changed 2 years ago by alfonsoml

Testcase

comment:3 Changed 2 years ago by alfonsoml

The sample file goes into the _samples folder and should be self-explainatory. Just remember to use IE.

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