Opened 9 years ago
Last modified 8 years ago
#16823 confirmed Bug
Tabletools and liststyle should not force custom context menu
| Reported by: | Marek Lewandowski | Owned by: | |
|---|---|---|---|
| Priority: | Nice to have (we want to work on it) | Milestone: | |
| Component: | General | Version: | |
| Keywords: | Cc: |
Description (last modified by )
Currently there's no way to load tabletools/liststyle plugin without adding a custom context menu. This is something that part of community would like to avoid.
Reason for this is that these plugins has a hard requirement on it, and this is how we need to proceed, because by default contextmenu is the only way how people can interact with it.
Change History (5)
comment:1 Changed 9 years ago by
| Status: | new → confirmed |
|---|---|
| Version: | 4.7.0 (GitHub - major) |
comment:2 Changed 9 years ago by
comment:3 Changed 9 years ago by
The solution that might work here is to add:
optionalproperty to CKEDITOR.pluginDefinition, so having plugin listed there it means that it's required, but optoinal.- Simply go with another configuration option like
config.contextmenu_disableand iftruecontextmenu plugin would be loaded but it's logic wouldn't kick in. - Extract logic from current plugins to e.g. tablettolsbase, liststylebase (to be consistent with embedbase) and then
tabletoolswould be only about bringing interface for the feature.
comment:4 Changed 9 years ago by
For others looking for a workaround the one that comes to my mind is simply... to stub contextmenu plugin.
So if you're building CKEditor using [presets builder](https://github.com/ckeditor/ckeditor-presets), make your fork where you add a single file to plugins as simple as that:
CKEDITOR.plugins.add( 'contextmenu', {} );
And that's all, then just go and build.sh your distribution.
---
For standard, uncompressed ckeditor-dev version, you simply need to replace whole plugin contextmenu plugin dir, leaving only this stub.
That's the workaround needed until this issue is solved.
comment:5 Changed 8 years ago by
| Description: | modified (diff) |
|---|

It has been requrested on SO: http://stackoverflow.com/questions/2246631/how-to-disable-ckeditor-context-menu
This issue has some story, similar issue was in #9284 however it came with a different result, by simply puting an error.