Opened 2 months ago

Last modified 2 months ago

#16983 confirmed Bug

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

Reported by: wwalc 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 j.swiderski 2 months ago.

Download all attachments as: .zip

Change History (4)

Changed 2 months ago by j.swiderski

comment:1 Changed 2 months ago by j.swiderski

  • Status changed from new to pending

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 2 months ago by j.swiderski (previous) (diff)

comment:2 Changed 2 months 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 2 months ago by j.swiderski

  • Status changed from pending to confirmed
  • Version set to 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 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy