Ticket #4652 (closed Bug: fixed)
Config for disable editor context menu
| Reported by: | garry.yao | Owned by: | tobiasz.cudnik |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.5 |
| Component: | UI : Context Menu | Version: | |
| Keywords: | Cc: |
Description
We should provide an option for disable context menu in editor, probably porting #311 from v2.
Attachments
Change History
comment:4 Changed 3 years ago by garry.yao
- Owner set to garry.yao
- Keywords Confirmed Review? added
- Status changed from new to assigned
'contextmenu' plugin is no longer required by 'menubutton' thus can be removed from plugin list to reveal the native context menu.
comment:5 Changed 3 years ago by alfonsoml
It seems to me that the overall idea and changes are OK, but due to the number of changes, I would like that someone else could look also at the patch.
comment:6 Changed 3 years ago by alfonsoml
- Keywords Review- added; Review? removed
There's some error with this patch:
in IE8, select some word and open the context menu, dismiss it, select something else and trying to open the context menu will throw an error. If the second time nothing is selected, the context menu opens, and when it's closed it selects back the first range.
comment:8 Changed 3 years ago by tobiasz.cudnik
- Status changed from review_failed to assigned
- Owner changed from garry.yao to tobiasz.cudnik
I'm taking this one over Garry.
comment:9 Changed 3 years ago by tobiasz.cudnik
- Status changed from assigned to review
Reason for non unlocked selection on IE was unexecuted onHide listener.
comment:11 Changed 3 years ago by garry.yao
- Status changed from review to review_failed
There's an error thrown when menu is hide at L145 of menu/plugin.js.
comment:14 Changed 3 years ago by tobiasz.cudnik
- Keywords Confirmed removed
- Status changed from review_passed to closed
- Resolution set to fixed
Fixed with [5937].
Actually here we just need to make it possible to disable it by simply remove the plugin itself, no new config introduced.