Opened 8 years ago

Closed 7 years ago

#4647 closed Bug (fixed)

Image info doesn't fill up dialog when open from context menu

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.2
Component: UI : Context Menu Version: SVN (CKEditor) - OLD
Keywords: IE Review+ Cc:

Description (last modified by Garry Yao)

  1. Open the 'replace by class' sample page in IE6, load the editor with following content and selection:
    	<p style="width:200px">[<img src="http://dev.fckeditor.net/chrome/site/logos.gif" />]</p>
    
  2. Right click to open context menu and select 'Image';
  • Actual Result: The 'Url' field is empty in opened dialog.

Attachments (1)

4647.patch (836 bytes) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 8 years ago by Garry Yao

Resolution: duplicate
Status: newclosed

Yet another DUP of #4041.

comment:2 Changed 8 years ago by Garry Yao

Milestone: CKEditor 3.1CKEditor 3.2
Resolution: duplicate
Status: closedreopened

It turns out that #4041 will not handle this bug, reopen it.

comment:3 Changed 8 years ago by Garry Yao

Manual TCs added at : http://ckeditor.t/tt/4647/1.html

comment:4 in reply to:  2 Changed 8 years ago by Garry Yao

Description: modified (diff)

Update missing pre-conditions in description.

Changed 8 years ago by Garry Yao

Attachment: 4647.patch added

comment:5 Changed 8 years ago by Garry Yao

Keywords: IE Review? added; IE6 removed

Affected all IE versions, the key to reproduce this bug is to have the selected element's container (paragraph here) has "layout", where re-selecting an image element will fail silently, result in a collapsed selection before the image.
The simplest way of avoid running into this issue is DO NOT forcing the selection into editor when context menu item is clicked.

comment:6 Changed 8 years ago by Garry Yao

#5053 has been marked as duplicate.

comment:7 Changed 8 years ago by Garry Yao

Owner: set to Garry Yao
Status: reopenednew

comment:8 Changed 8 years ago by Garry Yao

Status: newassigned

#4749 has been marked as duplicate.

comment:9 Changed 7 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:10 Changed 7 years ago by Garry Yao

Component: UI : DialogsUI : Context Menu
Resolution: fixed
Status: assignedclosed

Fixed with [5006].

comment:11 Changed 7 years ago by Pablo Vanwoerkom

Milestone: CKEditor 3.2CKEditor 3.x
Priority: NormalHigh
Resolution: fixed
Status: closedreopened

This is still a problem unfortunately. I just copied the sample html into the source in the demo page (ckeditor.com/demo) and the first time I right click the image I don't get the "Image Properties" option. However, if I then right-click on the image again OR if I left click the image and then click the Image button, I get the "Image Properties" option and the dialog loads correctly.

If it matters, I am running IE8 (32-bit) in Windows 7 (64-bit).

comment:12 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.xCKEditor 3.2
Priority: HighNormal
Resolution: fixed
Status: reopenedclosed

Please do not reopen tickets that have been already fixed. Maybe we have the regression because of something else that needs to be checked.

I would ask you to please open a new ticket for this issue, including all the necessary information to reproduce the problem and eventually a reference to this ticket.

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