For anyone how has optomized their site and included all the scripts just before the tag, the module seems to append Shadowbox.init to $page_bottom, before the jQuery libray is loaded. Only quick solution I see is to hard code it into html.tpl.

Shadowbox.init shoud be appended to $scripts variable instead..

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Danny Englander’s picture

I just ran into this issue as well, I too have my <?php print $scripts; ?> tag right before the closing </body> tag so my Shadowbox is breaking. if I move the scripts tag back to , shadowbox works fine.

There's a lot of advantages to having the scripts tag at the end so it would be nice to have Shadowbox play along. I guess this is not as much of a bug report as a feature request then but I won't change the category yet.

manfer’s picture

Status: Active » Needs review
FileSize
463 bytes
Danny Englander’s picture

Thanks, the patch worked against this version.

manfer’s picture

Version: 7.x-3.0-beta8 » 7.x-4.x-dev
Status: Needs review » Needs work

Pushed on 7.x-3.x version. Same needed in 7.x-4-x version

manfer’s picture

Status: Needs work » Needs review
FileSize
462 bytes
manfer’s picture

Status: Needs review » Fixed

Pushed on 7.x-4.x

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit 543ff82 on 7.x-4.x, 8.x-1.x by manfer:
    Issue #1659716 by manfer: Fixed javascript optimization problem