Opened 6 years ago

Closed 5 years ago

#11146 closed Bug (fixed)

[IE] Preview window switches IE to QM

Reported by: Piotr Jasiun Owned by: Marek Lewandowski
Priority: Normal Milestone: CKEditor 4.3.2
Component: General Version: 4.3
Keywords: Cc:

Description (last modified by Marek Lewandowski)

  1. Open Image2 sample.
  2. Press "preview" button.

Captions are not under but next to images. That's because preview loads in QM.


Solving this issue may fix also ticket #10762

Change History (12)

comment:1 Changed 6 years ago by Piotr Jasiun

Summary: [IE11]Image2 preview[IE11] Image2 preview

comment:2 Changed 6 years ago by Piotrek Koszuliński

Description: modified (diff)
Status: newconfirmed
Summary: [IE11] Image2 preview[IE] Image2 preview

comment:3 Changed 6 years ago by Olek Nowodziński

Milestone: CKEditor 4.3.1

comment:4 Changed 6 years ago by Piotrek Koszuliński

Description: modified (diff)
Summary: [IE] Image2 preview[IE] Preview window switches IE to QM

comment:5 Changed 6 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 4.3.1CKEditor 4.3.2

comment:6 Changed 6 years ago by Marek Lewandowski

Owner: set to Marek Lewandowski
Status: confirmedassigned

comment:7 Changed 6 years ago by Marek Lewandowski

Description: modified (diff)

comment:8 Changed 6 years ago by Marek Lewandowski

This issue will require me to do general fix in preview plugin. It will also fix following issues:

#10762 - IE - preview displays JS code in URL bar
#11323 - [FF] preview does not display images
#11248 - [FF]: Permission denied is thrown when preview is used for page with document.domain

comment:9 Changed 6 years ago by Marek Lewandowski

Status: assignedreview

After 3 days of my research and 5 minutes of Freds research he came with solution that window.location works diffrently than window.open argument.

Result is in branch t/11146b at dev.

comment:10 Changed 5 years ago by Olek Nowodziński

Rebased t/11146b on latest master.

comment:11 Changed 5 years ago by Olek Nowodziński

Status: reviewreview_passed

I pushed a commit to t/11146b which solves the issue in IE8. It turned out that it needs CKEDITOR.tools.enableHtml5Elements to display the preview properly.

#10762 - solved (also R+)
#11323 - broken
#11248 - broken

comment:12 Changed 5 years ago by Piotrek Koszuliński

Resolution: fixed
Status: review_passedclosed

Fixed on master with git:6e69ed5.

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