Opened 8 years ago
Closed 7 years ago
#4825 closed Bug (fixed)
Browser context menu is opened when clicking right mouse button twice
| Reported by: | Wiktor Walc | Owned by: | Minh Nguyen |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.3 |
| Component: | General | Version: | 3.0.1 |
| Keywords: | Confirmed Review+ | Cc: |
Description
This issue can be reproduced in any browser (tested in IE8, Safari 4.0.4, Opera 10.10, FF 3.5.5), but:
- in FF 3.5.5 @ XP it doesn't happen always, however it happens often when right-clicking twice on a link
In FCKeditor I'm having this problem only in Opera.
Attachments (3)
Change History (13)
comment:1 Changed 8 years ago by
| Summary: | Browser context menu is opened when clicking mouse button twice → Browser context menu is opened when clicking right mouse button twice |
|---|
comment:2 Changed 8 years ago by
| Milestone: | → CKEditor 3.3 |
|---|
Changed 7 years ago by
| Attachment: | 4825.patch added |
|---|
comment:3 Changed 7 years ago by
| Keywords: | Review? added |
|---|---|
| Owner: | set to Minh Nguyen |
| Status: | new → assigned |
comment:4 Changed 7 years ago by
| Keywords: | Review- added; Review? removed |
|---|
It's a nice approach (actually also used in IE and FF), while the offset must be menu orientation sensitive, e.g. when menu is layout toward north (when opened at the bottom of the window), the justification must be applied as negative value.
comment:5 Changed 7 years ago by
Considering that the menu is opened inside our stuff, aren't we able to block the context menu event instead of proposing a hack-like approach?
comment:6 Changed 7 years ago by
| Keywords: | Review? added; Review- removed |
|---|
comment:7 Changed 7 years ago by
| Keywords: | Review- added; Review? removed |
|---|
Such logic is already [http://dev.fckeditor.net/browser/CKEditor/trunk/_source/plugins/panel/plugin.js#L297
planted], we should make it works instead of repeating in another places.
Changed 7 years ago by
| Attachment: | 4825_3.patch added |
|---|
comment:8 Changed 7 years ago by
| Keywords: | Review? added; Review- removed |
|---|
Such logic is already http://dev.fckeditor.net/browser/CKEditor/trunk/_source/plugins/panel/plugin.js#L297
This line code using for prevent in content of context menu, while we need prevent the root block of context menu.

With the first right click we will show context menu, then second right click the mouse active on context menu (just show) not on editor. This is reason the default context menu show when we have twice right click. My approach only moves context menu a bit far from mouse's position.