Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This error shows up in Chrome if the effect None is chosen. This seems to be an issue with the latest version of JAIL.js. Here is a patch to fix this, it's also attached to this issue:
diff -rupN jail_old/jail.module jail/jail.module
--- jail_old/jail.module 2011-05-04 22:06:06.000000000 +0200
+++ jail/jail.module 2011-11-17 18:05:08.000000000 +0100
@@ -150,7 +150,13 @@ function theme_jail_formatter($variables
$output .= '<noscript><img src="' . $url . '"></noscript>';
}
drupal_add_js(libraries_get_path('jail') . '/jquery.jail.js');
- drupal_add_js('(function ($) {$(".lazy").jail({placeholder : "/' . variable_get('jail_loader', '/' . drupal_get_path('module', 'jail') . '/white.gif') . '", effect : "' . $variables['jail_effect'] .'"});})(jQuery);', array('type' => 'inline', 'scope' => 'footer', 'weight' => 5));
+
+ $js = '(function ($) {$(".lazy").jail({placeholder : "/' . variable_get('jail_loader', '/' . drupal_get_path('module', 'jail') . '/white.gif') . '"';
+ if($variables['jail_effect'] != "none") {
+ $js .= ', effect : "' . $variables['jail_effect'] .'"';
+ }
+ $js .= '});})(jQuery);';
+ drupal_add_js($js, array('type' => 'inline', 'scope' => 'footer', 'weight' => 5));
return $output;
}
Comment | File | Size | Author |
---|---|---|---|
jail_effects.patch | 1.03 KB | elmotri |
Comments
Comment #1
danlinn CreditAttribution: danlinn commentedAwesome, thanks! I'll roll this into the next release.
Comment #2
danlinn CreditAttribution: danlinn commentedComment #3
danlinn CreditAttribution: danlinn commentedI've rolled a new version. Should be available soon. Thanks for your help!