Opened 15 years ago
Closed 15 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 15 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 15 years ago by
Milestone: | → CKEditor 3.3 |
---|
Changed 15 years ago by
Attachment: | 4825.patch added |
---|
comment:3 Changed 15 years ago by
Keywords: | Review? added |
---|---|
Owner: | set to Minh Nguyen |
Status: | new → assigned |
comment:4 Changed 15 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 15 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 15 years ago by
Keywords: | Review? added; Review- removed |
---|
comment:7 Changed 15 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 15 years ago by
Attachment: | 4825_3.patch added |
---|
comment:8 Changed 15 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.