Opened 7 years ago

Closed 7 years ago

#6404 closed Bug (fixed)

Contextual menu doesn't disappear when switch in HTML source view

Reported by: fournaise Owned by: Martin
Priority: Normal Milestone: CKEditor 3.4.3
Component: UI : Context Menu Version: 3.3.1
Keywords: IBM Cc: satya_minnekanti@…

Description

Tested on IE8, FF3.6, CH6 under WinXP. When you are in edition view, open a contextual menu (mouse right click) and after switch to HTML source view, the contextual menu stays open.

Attachments (2)

6404.patch (745 bytes) - added by Martin 7 years ago.
6404_2.patch (672 bytes) - added by Martin 7 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 7 years ago by Krzysztof Studnik

Component: GeneralUI : Context Menu
Status: newconfirmed
Version: 3.4.2 (SVN - trunk)3.3.1

In CKEditor 3.3 context menu hides itself when switching between modes. In 3.3.1+ menu stays open

comment:2 Changed 7 years ago by Satya Minnekanti

Cc: satya_minnekanti@… added
Keywords: IBM added

comment:3 Changed 7 years ago by Martin

Owner: set to Martin
Status: confirmedassigned

Changed 7 years ago by Martin

Attachment: 6404.patch added

comment:4 Changed 7 years ago by Martin

Status: assignedreview

comment:5 Changed 7 years ago by Garry Yao

Status: reviewreview_failed

There're some chaos in the patch while it's logically right.

comment:6 Changed 7 years ago by Martin

what you mean 'chaos', colud you explain what is wrong in patch ?

comment:7 Changed 7 years ago by Sa'ar Zac Elias

I believe he meant L5 that for some reason was removed, coding style issues and missing semicolon.

Changed 7 years ago by Martin

Attachment: 6404_2.patch added

comment:8 Changed 7 years ago by Martin

Status: review_failedreview

comment:9 Changed 7 years ago by Wiktor Walc

Milestone: CKEditor 3.5.1

comment:10 Changed 7 years ago by Sa'ar Zac Elias

Milestone: CKEditor 3.5.1CKEditor 3.4.3
Resolution: fixed
Status: reviewclosed

Fixed using the 3.4.3, probably by [6095].

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