Opened 8 years ago
Last modified 7 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 8 years ago by
Status: | new → confirmed |
---|---|
Version: | 4.7.0 (GitHub - major) |
comment:2 Changed 8 years ago by
comment:3 Changed 8 years ago by
The solution that might work here is to add:
optional
property 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_disable
and iftrue
contextmenu 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
tabletools
would be only about bringing interface for the feature.
comment:4 Changed 8 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 7 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.