Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

Doc, add ngTouch as optional dependency #5499

Closed
RopoMen opened this issue Feb 17, 2016 · 1 comment
Closed

Doc, add ngTouch as optional dependency #5499

RopoMen opened this issue Feb 17, 2016 · 1 comment

Comments

@RopoMen
Copy link
Contributor

RopoMen commented Feb 17, 2016

Carousel is using ng-swipe-right and ng-swipe-left which are from ngTouch module
https://github.com/angular-ui/bootstrap/blob/master/template/carousel/carousel.html

https://docs.angularjs.org/api/ngTouch/directive/ngSwipeRight

Readme.md contains this under "Installation"
"Note: Since version 0.13.0, UI Bootstrap depends on ngAnimate for transitions and animations, such as the accordion, carousel, etc. Include ngAnimate in the module dependencies for your app in order to enable animation."

I think it should also mention that ngTouch is required if you need swipe to work with carousel.

Br,
Markku

@wesleycho
Copy link
Contributor

Feel free to file a PR for updating these docs.

@wesleycho wesleycho added this to the Backlog milestone Feb 17, 2016
deeg added a commit to deeg/bootstrap that referenced this issue Feb 17, 2016
Note that the carousel requires ngTouch for swiping.

Closes angular-ui#5499
deeg added a commit to deeg/bootstrap that referenced this issue Feb 17, 2016
Note that the carousel requires ngTouch for swiping.

Closes angular-ui#5499
Closes angular-ui#5500
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants