It was reported that the position of the main script was causing issues with earlier versions of IE browsers.

By request it was asked that the main section of the script be placed in the header while the init functions remain in the footer.

I did some quick work to add an option to the module to allow the positioning of the main part of of the meebo script to be either in the header or the footer.

Here's a patch that can add the functionality in to the module. For your consideration.

CommentFileSizeAuthor
#2 meebo-js_position-1462034-2.patch4.43 KBRok Žlender
drupal-meebo_js_pos_option.patch2.43 KBcntlscrut

Comments

Rok Žlender’s picture

Rok Žlender commented
Status: Active » Needs work

I think your patch is missing

drupal_add_js($script, array('type' => 'inline', 'scope' => 'footer'));

in else.

Rok Žlender’s picture

Rok Žlender commented
StatusFileSize
new meebo-js_position-1462034-2.patch4.43 KB

New patch which fixes my above comment.

Rok Žlender’s picture

Rok Žlender commented
Status: Needs work » Needs review