Opened 2 years ago

Last modified 20 months 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 Marek Lewandowski)

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 2 years ago by Marek Lewandowski

Status: newconfirmed
Version: 4.7.0 (GitHub - major)

comment:2 Changed 2 years ago by Marek Lewandowski

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.

comment:3 Changed 2 years ago by Marek Lewandowski

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 if true 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.
Last edited 2 years ago by Marek Lewandowski (previous) (diff)

comment:4 Changed 2 years ago by Marek Lewandowski

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 20 months ago by Marek Lewandowski

Description: modified (diff)
Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy