Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gridmanager throws error with default options and tinyMCE #67

Open
madc opened this issue Jul 29, 2015 · 2 comments
Open

Gridmanager throws error with default options and tinyMCE #67

madc opened this issue Jul 29, 2015 · 2 comments
Labels

Comments

@madc
Copy link

madc commented Jul 29, 2015

The gridmanager throws the following error, when tinyMCE is present on the page and tinymce.config is not overwritten.

TypeError: t.plugins.split is not a function

The problem here seems to be the default optinos for tinyMCE in gridmanager.js on line 1527. Here, tinymce.config.plugins is preset with an array instead a string. Removing [ and ] should solve this issue.

@neokoenig
Copy link
Owner

Which version of TinyMCE? I've only tested against 4.0

@madc
Copy link
Author

madc commented Jul 29, 2015

The TinyMCE version is 4.0.12, jQuery version is 1.11.0.
In the documentation it says, TinyMCE expects the plugins to be "a comma or space seperated string, or an array of strings".

I just saw, that it is working on your demo, so maybe this is a problem with running TinMCE < 4.1?
Anyway removing the brackets should have no effect on newer versions of the editor and fix it for older ones.

@neokoenig neokoenig added the bug label Jul 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants