#5799 closed Bug (fixed)
liststyle plugin doesn't state required plugins
| Reported by: | Alfonso Martínez de Lizarrondo | Owned by: | Alfonso Martínez de Lizarrondo |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.4 |
| Component: | General | Version: | 3.3 |
| Keywords: | Cc: |
Description
The liststyle plugin doesn't have a "requires" statement about what plugins it needs, and although it tries to detect the availability of the contextMenu, the editor.addMenuGroup isn't protected.
http://cksource.com/forums/viewtopic.php?f=11&t=18990
The code could be changed also to state that the contextmenu is required and remove the "if"s in the code, but I think that this way it can allow for other people for example to create a toolbar button for them even if they disable the contextmenu.
Attachments (1)
Change History (4)
Changed 14 years ago by
| Attachment: | 5799.patch added |
|---|
comment:1 Changed 14 years ago by
| Milestone: | → CKEditor 3.4 |
|---|---|
| Status: | review → review_passed |
comment:2 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [5732]
Note: See
TracTickets for help on using
tickets.
Proposed patch