search

OdooCommunityWidgets / website_multi_image

Odoo Website Multi-Image Module
MIT License
29 stars 34 forks source link

fix #22 #23

Closed a0c closed 9 years ago

a0c commented 9 years ago

in synced sliders, use natural ordering of slides instead of index attribute to navigate with mouse clicks, cos filtering could be applied

lukebranch commented 9 years ago

@a0c ,

Would you mind doing a PR against:

https://github.com/kenwheeler/slick

As this is a third-party project (slickjs) i'd prefer not modifying core code in that project if a fix can be merged further upstream with the original SlickJS project, as this will help make it easier to maintain and upgrade.

If they will not merge it in the main slickJS project then i'll look at merging it here.

a0c commented 9 years ago

@lukebranch, Already done. And only duplicated the PR here, because you don't seem to use the latest version of Slick, do you? 1.3.15 vs 1.4.1

lukebranch commented 9 years ago

@a0c ,

Thank you for clarifying that, I have not updated the project in awhile due to time constraints.