-
Notifications
You must be signed in to change notification settings - Fork 123
Requirements for installation are not clear #73
Comments
Yes, it is implied that bootstrap slider is already included. If you'd be willing to submit a PR on that, I'd take that. You are correct, this does look for |
I second both points, it would be great if the actual installation hints would be complete. When using it without jQuery, I see an |
That's strange. In any event, if someone really needs the jQuery dependency to be removed, feel free to submit a PR. I suppose I'll flesh out the docs a bit more to say you do need to include bootstrap-slider (though I still feel this is a given). |
Sorry guys, but i'm not understand what are the requirements. |
Yes, you need to install bootstrap-slider. That is implied when I say:
and
bower should already install |
Thank you for your fast response, http://plnkr.co/edit/wumvDyTDAZiG4P2KlsLQ?p=preview Coul'd you check if I miss something? |
Yes, you forgot to require the module |
Concerning ui.bootstrap-slider you mean the Angular UI Bootstrap or jQuery ones? http://angular-ui.github.io/bootstrap/ Could you kindly write me the right ui.bootstrap-slider project page (or bower or CDN link) so i'll try to add into my sample. |
... You literally just have to add it as a module dependency, like so: (function () {
var app = angular.module("sliderApp", ['ui.bootstrap-slider']);
app.controller('sliderCtrl', function($scope, $http) {
this.sliderValue = 24;
});
})(); You can see here that that is what I've chosen to call this module. |
Oh damn, |
This is pretty obvisous, but I guess, people might forget about the module inclusion at first (I did). |
I actually had to include
seiyria-bootstrap-slider/dist/bootstrap-slider.min.js
andseiyria-bootstrap-slider/dist/css/bootstrap-slider.min.css
too.Also, looking at the code, I'm not sure it works without jQuery, like the original plugin is supposed to. It would really be a plus, since most angular modules for UI still require it...
The text was updated successfully, but these errors were encountered: