Opened 6 years ago

Last modified 6 years ago

#11506 confirmed Bug

Image inside floating DIV looses selection with Context Menu on IE

Reported by: Jose Flores Owned by:
Priority: Normal Milestone:
Component: UI : Context Menu Version: 4.0
Keywords: IE Cc:

Description

Descriptive summary: When you move the focus out of the image area and then do a right clic on the image, Image Properties option is not displayed in the context menu

  • I just erased a very detailed steps to reproduce and other information because akismet rejected my submit lots of times -

Attachments (3)

contextmenu.patch (497 bytes) - added by Jose Flores 6 years ago.
win8 IE10.png (51.4 KB) - added by Jakub Ś 6 years ago.
Win7 IE11.png (52.7 KB) - added by Jakub Ś 6 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 6 years ago by Jose Flores

Descriptive summary: When you move the focus out of the image area and then do a right clic on the image, Image Properties option is not displayed in the context menu

Steps to reproduce:

  1. Go to the full version of ckeditor demo
  2. Delete all the content and paste this into the source code editor:
    <div style="float: left;"><img src="http://a.cksource.com/e/1/img/logo-ckeditor-h100.png" /></div>
    
  1. Go to the wysiwyg editor and clic on the image to select it
  2. Move the focus out of the image: eg: press the arrow right key to move the focus/cursor
  3. Right clic on the image

Expected: Image Properties option should be displayed on the context menu
Actual: Image Properties option is not displayed, only Paste option

Browser and OS: It only happens on IE. (Chrome, Safari and Firefox seems to work fine). I had only chance to see this behaviour on IE9, IE10 and IE11

Other: There is an old ticket which describes the same bug/behaviour: http://dev.ckeditor.com/ticket/798 and there was a fix for that but in the predecessor of CKEditor, FCKEditor. I have verified this in the ckeditor demo and also the "Nightly Build" version.

Last edited 6 years ago by Jose Flores (previous) (diff)

Changed 6 years ago by Jose Flores

Attachment: contextmenu.patch added

comment:2 Changed 6 years ago by Jose Flores

I have attached a possible patch which works for me (tested on IE9, IE10 and IE11). Remaining tasks would be test it on previous versions of IE that you support and of course validate if this change it's ok for you. The patch is over the contextmenu plugin https://github.com/ckeditor/ckeditor-dev/blob/master/plugins/contextmenu/plugin.js#L98.

By the way, this is pretty similar to the patch for the same issue on the old FCKEditor in http://dev.ckeditor.com/changeset/2375 I hope this can be useful for you.

Last edited 6 years ago by Jose Flores (previous) (diff)

comment:3 Changed 6 years ago by Jakub Ś

Keywords: IE added; image contextmenu ie removed
Status: newconfirmed
Version: 4.3.14.0

Problem can be reproduced from CKEditor 4.0 in IE9+

What you need to do is:

  1. Switch to source and paste provided code
  2. Switch to wysiwyg and left-click on image
  3. Press right arrow
  4. Right-click

After step 3 whole image will be selected, there will be native border around it (Please see win8 IE10.jpg).
In IE11 this problem can only be reproduced from CKEditor 4.3. When you click on image (Step 2), not only the border shows but also visible selection on image. This selection could only be seen for a split of second in older versions of editor (Please see win7 IE11.jpg).

Changed 6 years ago by Jakub Ś

Attachment: win8 IE10.png added

Changed 6 years ago by Jakub Ś

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