Opened 10 years ago

Closed 10 years ago

#4826 closed Bug (fixed)

Context menu doesn't work sometimes

Reported by: Wiktor Walc Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: UI : Context Menu Version: SVN (CKEditor) - OLD
Keywords: Confirmed 3.0.2 Review+ Cc:

Description (last modified by Wiktor Walc)

This is a regression bug, this problem didn't exist in 3.0.1. I don't see any errors in the error console when this bug occurs.

Steps to reproduce

Using Safari 4.0.4 @ Win (XP/SP3 in my case)

  1. restart Safari
  2. open page with CKEditor (trunk, 3.1 branch, doesn't matter)
  3. right click your mouse to open the context menu
  4. repeat steps 1-3 until you'll see that context menu is not working

Instead of restarting, you may also try opening multiple tabs until you'll find that one with context menu not working.

Once you'll be able to open Safari in "CKEditor context menu not working" mode, you can try clearing the browser cache, refreshing the page and so on and the context menu will be still broken in that tab. If you open CKEitor 3.0.1 in the same tab, context menu will be working there (in CKEditor 3.0.1).

One user reported that on Snow Leopard sontext menu doesn't work in Safari, Google Chrome beta and FF 3.5.3. Another Snow Leopard user reported that this issue happens randomly, just as I described above and only in Safari. So, probably this is issue only exists in Safari, I don't have a Mac to check it.

Attachments (2)

4826.patch (1.4 KB) - added by Garry Yao 10 years ago.
4826_2.patch (1.3 KB) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 10 years ago by Wiktor Walc

Description: modified (diff)

comment:2 Changed 10 years ago by Wiktor Walc

Milestone: CKEditor 3.1

comment:3 Changed 10 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned

Confirmed on Safari 4.0.4, XP3 that 'domEvent.ctrlKey' is always 'true' on 'contextmenu' event, one possible solution here would be disable the 'nativeContextMenuOnCtrl' config for this specific environment.

comment:4 Changed 10 years ago by Garry Yao

Component: GeneralUI : Context Menu

Regression of [4540].

Changed 10 years ago by Garry Yao

Attachment: 4826.patch added

comment:5 Changed 10 years ago by Garry Yao

Keywords: Review? added

Please help me to confirm that the fixing is well targeted (Safari + Windows).

Changed 10 years ago by Garry Yao

Attachment: 4826_2.patch added

comment:6 Changed 10 years ago by Garry Yao

Targeting both Windows and Mac.

comment:7 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:8 Changed 10 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: assignedclosed

Fixed with [4748].

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