Skip to content

Fixed Mootools Conflics#133

Open
thexmanxyz wants to merge 3 commits intobooncon:masterfrom
thexmanxyz:master
Open

Fixed Mootools Conflics#133
thexmanxyz wants to merge 3 commits intobooncon:masterfrom
thexmanxyz:master

Conversation

@thexmanxyz
Copy link
Copy Markdown

@thexmanxyz thexmanxyz commented Nov 18, 2017

Hey there,

I noticed a bug which doesn't allow Slippry to coexist with the Mootools library. Especially with either mootools-core.js or mootools-more.js. However I needed both Slippry and Mootools to be on the same page in one of my projects so I decided to create a pull request which reflects my changes which were necessary to get it to work correctly. The problem originates in the load event which is triggered on preload of the slider images. This somehow causes serious issues with Mootools. One important reference I have to make is an other pull request of another repository which pointed me in the right direction #53 - Collage Plus.

Basically I simply replaced the default load event with a custom one called loads (s for slippery). With the proposed changes it now works without any issues.

Edit: Updated image load trigger to react on cached and non cached page load.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant