Opened 10 years ago
Last modified 10 years ago
#12716 confirmed New Feature
TableTools should be made available from toolbar when context menu isn't used.
Reported by: | Jakub Ś | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | 4.0 |
Keywords: | Cc: |
Description
We have made a fix in #9284 but now users can't use tabletools if they don't want to use contextmenu plugin.
We could make tabletools available from toolbar but first we need to fix: http://dev.ckeditor.com/ticket/7903.
Change History (6)
comment:1 Changed 10 years ago by
Status: | new → confirmed |
---|
comment:3 Changed 10 years ago by
Thank you for checking this one. Please note however that we first need to fix #7903.
- This is indeed a problem and should be fixed.
- Yes, elements should be disabled instead of being hidden.
If possible please don't remove this fiddle or please attach your sample code to this ticket for future reference.
comment:5 Changed 10 years ago by
- How should it work? I guess this way:
if ( editor.contextMenu ) { // Add context menu items } else { // add toolbar button }
Right?
- We need icons. At least one icon for toolbar button. Can we ask moono designer about it?
comment:6 Changed 10 years ago by
- How should it work? I guess this way:
The right way would be to first solve #7903 to have a public API. Then extracting part of tabletools which creates the context menu options to a separate plugin (e.g. tabletoolscontextmenu). And then creating new plugin tabletoolsbutton. This way everyone will be able to choose which option she/he prefers. For example, one may want to use the contextmenu plugin, but still move the table tools from the context menu to the toolbar.
- We need icons. At least one icon for toolbar button. Can we ask moono designer about it?
The moono's icons author is available, so as soon as there's something to work on, we can ask him.
Looks like moving menu items from context menu to the toolbar menu discovers another issues: http://jsfiddle.net/danya_postfactum/gwragLhw/