search

jsor / jcarousel

Riding carousels with jQuery.
https://sorgalla.com/jcarousel/
MIT License
1.99k stars 736 forks source link

Document hasPrev(), hasNext() #646

Closed cburgmer closed 10 years ago

cburgmer commented 10 years ago

We are using hasPrev() & hasNext() to find out about the scrolling state of our carousel. This is however not documented at all and we wonder whether we might be using an internal interface. This of course could mean that our code will break even on minor version upgrades.

We know about the existence of the jCarousel control plugin, however the aync nature of the control update renders it unfit for our use case (we would need to wait for 3 events to be fired in sequence createdend, inactive & active).

So we'd be happy if both methods could be supported in the future in some form.

jsor commented 10 years ago

Until now, these methods are not considered part of the pubilc API. But i see it's usefulness, so feel free to submit a PR and add the methods to https://github.com/jsor/jcarousel/blob/master/docs/reference/api.md. Once in the API docs, it will not change in minor version upgrades.