Opened 10 years ago

Closed 10 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 10 years ago by Piotr Jasiun

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

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

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

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

Milestone: CKEditor 4.3.1

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

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

comment:5 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 4.3.1CKEditor 4.3.2

comment:6 Changed 10 years ago by Marek Lewandowski

Owner: set to Marek Lewandowski
Status: confirmedassigned

comment:7 Changed 10 years ago by Marek Lewandowski

Description: modified (diff)

comment:8 Changed 10 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 10 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 10 years ago by Olek Nowodziński

Rebased t/11146b on latest master.

comment:11 Changed 10 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 10 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 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy