Opened 10 years ago

Last modified 9 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 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 10 years ago by Marek Lewandowski

Status: newconfirmed
Version: 4.7.0 (GitHub - major)

comment:2 Changed 10 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 10 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 10 years ago by Marek Lewandowski (previous) (diff)

comment:4 Changed 10 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 9 years ago by Marek Lewandowski

Description: modified (diff)
Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy