This patch fixes things like window.onload = function() ... to $(document).ready(function() ...); The only event I was unable to change to jQuery is the onselectstart as jQuery does not provide an event for that, only for select. This does not seem to fire anyway in my tests, but that is a different issue.

This patch is mainly to address these event issues so the module does not conflict with other modules. Using the jQuery api speeds up loading of the images quite a bit as it doesn't have to wait for window.onload to fire. Any suggestions, feedback, and improvements are welcome. I have an additional patch in case people want to link the images to their respective nodes and set that up in the view if anyone is interested.

Lucian Hontau
GoingOn | http://www.goingon.com/

CommentFileSizeAuthor
imageflow.js_.patch1.58 KBlucian303
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

riote’s picture

May be a silly question but how exactly does one apply the patch?

jamesbenison’s picture

Status: Needs review » Closed (won't fix)

Closing out obsolete issue.