Opened 7 years ago

Closed 7 years ago

#5586 closed Bug (fixed)

Multiple editor instances ruins full-screen mode

Reported by: Johan Fredrik Varen Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.3
Component: General Version: 3.2.1
Keywords: Confirmed Firefox Review+ Cc:

Description

By running two or more instances of the editor on the same page, swithing to full-screen mode in any of the instances, except the first instance, will ruin the full-screen mode (see attached screenshot). The bug can easily be reproduced by adding another instance in the "replacebyclass.html" example, focusing the editor field of that instance and hitting its full-screen button.

Attachments (2)

Multiple editors.png (54.9 KB) - added by Johan Fredrik Varen 7 years ago.
5586.patch (683 bytes) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (9)

Changed 7 years ago by Johan Fredrik Varen

Attachment: Multiple editors.png added

comment:1 Changed 7 years ago by Garry Yao

Keywords: Pending added
Milestone: CKEditor 3.3

WFM in our sample page.

comment:2 Changed 7 years ago by Johan Fredrik Varen

Milestone: CKEditor 3.3

Should have mentioned that this bug only occurs in Firefox (tested in version 3.6 on Mac and Windows XP). Works in IE 7 and 8, as well as Google Chrome.

The bug also occurs in the sample page in the previous comment.

comment:3 Changed 7 years ago by Alfonso Martínez de Lizarrondo

Keywords: Confirmed Firefox added; Pending removed
Milestone: CKEditor 3.3CKEditor 3.4

comment:4 Changed 7 years ago by Garry Yao

Milestone: CKEditor 3.4CKEditor 3.3
Owner: set to Garry Yao
Status: newassigned

Reproducible only when the second editor is focused.

Changed 7 years ago by Garry Yao

Attachment: 5586.patch added

comment:5 Changed 7 years ago by Garry Yao

Keywords: Review? added

Regression of [5353] where the focusGrabber element is breaks the layout with unknown reason.

comment:6 Changed 7 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

That's weird.... but the fix simplify things.

comment:7 Changed 7 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [5468].

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