Opened 8 years ago

Last modified 8 years ago

#16983 confirmed Bug

Maximize does not work s expected when classic and inline editors are mixed on the same page

Reported by: Wiktor Walc Owned by:
Priority: Normal Milestone:
Component: General Version: 4.0
Keywords: Cc:

Description

Originally reported on Disqus, did not try to reproduce it. http://ckeditor.com/addon/maximize#comment-3256559859

I was having everything in my page turned contenteditable when using maximize. Finaly I solved the issue. If you have a classic ckeditor and one or more inline editors in the same page, the maximize "refreshCursor" function only checks that the editor being maximized is not an inline one, but it does not check on the rest of CKEDITOR.instances, which are also refreshed..

The function's loop should be modified to:

for ( var i in all ) {
var one = all[ i ]; 
if ( !one.editable().isInline() && one.mode == 'wysiwyg' && !one.readOnly ) {
.........
}
}
]}}

Attachments (1)

maxtest.html (6.7 KB) - added by Jakub Ś 8 years ago.

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by Jakub Ś

Attachment: maxtest.html added

comment:1 Changed 8 years ago by Jakub Ś

Status: newpending

Please use the attached sample - it needs to be put into CKEditor samples folder.

The problem which I have noticed is that when you have focus in inline editor and press Maximize in iframed editor, the inline toolbar is still visible and it is possible to work in inline editor. I don't think however this is what author had in mind.

Last edited 8 years ago by Jakub Ś (previous) (diff)

comment:2 Changed 8 years ago by eresina

Hi, yes I still get the same problem with your maxtest.html attached sample. Everything becomes editable after using maximize. I can edit all the text outside the editors after maximizing in and out. But after trying with different browsers I've realized that I only get this problem with Firefox (using 52.1.0 build). I have not been able to reproduce it with Chrome, Opera or Safari.

comment:3 Changed 8 years ago by Jakub Ś

Status: pendingconfirmed
Version: 4.0

Yes, you are correct. I have checked the problem one more time and in deed everything is editable.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy