Inclusion of asset_tinymce_helper.js breaks collapsible menus when JQuery is enabled.

APR-02 distibution, have tried all the jQuery update versions as well.

More later if I can diagnose.

CommentFileSizeAuthor
#4 asset_tinymce_helper.js_3.patch631 bytesibot
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

1kenthomas’s picture

NOTE: took a quick look at this. Helper code is old and looks to be incompatible with JQuery_update (not simply JQuery). More later...

wmostrey’s picture

Title: Breaks JQuery collapsible menus on TinyMCE pages » Collapsible menus break on TinyMCE pages with jquery_update
ibot’s picture

Seems to me like an problem with the updated tinyMCE-API,
see http://drupal.org/node/215580

My collapsible-Menus are not workin anymore because of this error (firebug output):

tinyMCE.getEditorId is not a function

[Break on this error]  if(tinyMCE.getEditorId(textarea_id) == null){
asset_tinymce_hel... (line 17)
ibot’s picture

Status: Active » Needs review
FileSize
631 bytes

This patch was working for me..

wmostrey’s picture

Hey ibot, I'm a bit confused about http://drupal.org/node/215580. What version of TinyMCE and the TinyMCE module are you using for this patch to work?

ibot’s picture

Hi wmostrey,
sorry for the confusion..
Today i can't figure out what Versions i used, but i will try to check it the next days..

Thanks for the cool module!

ib0t

wmostrey’s picture

Thanks, it's just that I know what to put in the readme. If I push this change, then everyone will have to upgrade their TinyMCE, so I'd better get this right from the first time :)

wmostrey’s picture

Status: Needs review » Postponed (maintainer needs more info)
wmostrey’s picture

Status: Postponed (maintainer needs more info) » Closed (duplicate)