chore(bower): change path to bower_components

Author: revolunet

.bowerrc

@@ -1,4 +1,4 @@
 {
-    "directory": "components",
+    "directory": "bower_components",
     "json": "bower.json"
 }

.gitignore

@@ -1,3 +1,4 @@
 *.DS_Store
 node_modules
-components
+bower_components
+coverage

Gruntfile.js

@@ -1,7 +1,9 @@
-/* global require, module, process */
+/* global require, module, process, __dirname */
 
 'use strict';
 
+var path = require('path');
+
 module.exports = function(grunt) {
 
   // Project configuration.
@@ -21,31 +23,39 @@ module.exports = function(grunt) {
         options: {
           port: 9999,
           hostname: '0.0.0.0',
-          base: '.',
-          keepalive: true
+          base: '.'
         }
       }
     },
     dirs: {
+      src: 'src',
       dest: 'dist'
     },
+    copy: {
+
+    },
+    autoprefixer: {
+      source: {
+        options: {
+          //browsers: ['last 2 version', '> 1%', 'android', 'chrome', 'firefox']
+        },
+        src: '<%= dirs.src %>/css/<%= pkg.name %>.css',
+        dest: '<%= dirs.dest %>/<%= pkg.name %>.css'
+      }
+    },
     concat: {
       options: {
         banner: '<%= meta.banner %>'
       },
       dist: {
-        src: ['src/*.js', 'src/**/*.js'],
+        src: ['<%= dirs.src %>/*.js', '<%= dirs.src %>/**/*.js'],
         dest: '<%= dirs.dest %>/<%= pkg.name %>.js'
-      },
-      ngMobile: {
-        src: ['lib/angular-mobile.js'],
-        dest: '<%= dirs.dest %>/angular-mobile.js'
       }
     },
     cssmin: {
       combine: {
         files: {
-          '<%= dirs.dest %>/<%= pkg.name %>.min.css': ['src/css/*.css']
+          '<%= dirs.dest %>/<%= pkg.name %>.min.css': ['<%= dirs.dest %>/<%= pkg.name %>.css']
         }
       }
     },
@@ -59,7 +69,7 @@ module.exports = function(grunt) {
       }
     },
     jshint: {
-      files: ['Gruntfile.js', 'src/*.js', 'test/unit/*.js'],
+      files: ['Gruntfile.js', '<%= dirs.src %>/*.js', 'test/unit/*.js'],
       options: {
         curly: false,
         browser: true,
@@ -81,69 +91,50 @@ module.exports = function(grunt) {
         }
       }
     },
-    // watch: {
-    //   files: '<config:jshint.files>',
-    //   tasks: 'default'
-    // },
     karma: {
-      test: {
-        options: {
-          reporters: ['dots'],
-          singleRun: true
-        }
+      options: {
+          // needed to use absolute path for some reason
+          configFile: path.join(__dirname, 'test', 'karma.conf.js')
       },
-      server: {
-        options: {
-          singleRun: false
-        }
+      unit: {
+          port: 7101,
+          background: true
       },
-      options: {
-        configFile: 'test/karma.conf.js'
+      continuous: {
+          singleRun: true
       }
     },
     watch: {
-      files: ['src/**'],
-      tasks: ['quickbuild']
+      dev: {
+        files: ['<%= dirs.src %>/**'],
+        tasks: ['build']
+      },
+      test: {
+        files: ['test/unit/**'],
+        tasks: ['karma:unit:run']
+      }
+      
     }
   });
 
-  // Load the plugin that provides the "jshint" task.
+  grunt.loadNpmTasks('grunt-autoprefixer');
+  grunt.loadNpmTasks('grunt-contrib-copy');
   grunt.loadNpmTasks('grunt-contrib-jshint');
-
-  // Load the plugin that provides the "concat" task.
   grunt.loadNpmTasks('grunt-contrib-concat');
-
-  // Load the plugin that provides the "uglify" task.
   grunt.loadNpmTasks('grunt-contrib-uglify');
-
   grunt.loadNpmTasks('grunt-contrib-cssmin');
-
   grunt.loadNpmTasks('grunt-contrib-connect');
-
-  // Load the plugin that provides the "watch" task.
   grunt.loadNpmTasks('grunt-contrib-watch');
-
-
-  // Default task.
-  grunt.registerTask('default', ['test']);
+  grunt.loadNpmTasks('grunt-karma');
 
   // Test tasks.
-  grunt.registerTask('test', ['jshint', 'karma:test']);
-  grunt.registerTask('test-server', ['karma:server']);
+  grunt.registerTask('test', ['jshint', 'karma:unit']);
 
   // Build task.
-  grunt.registerTask('build', ['concat', 'uglify', 'cssmin', 'test']);
-  grunt.registerTask('quickbuild', ['jshint', 'concat', 'uglify', 'cssmin']);
+  grunt.registerTask('quickbuild', ['jshint', 'concat', 'uglify', 'autoprefixer', 'cssmin']);
+  grunt.registerTask('build', ['quickbuild', 'test']);
 
-  // run devserver
-  grunt.registerTask('webserver', ['connect:devserver']);
-
-  // Provides the "karma" task.
-  grunt.registerMultiTask('karma', 'Starts up a karma server.', function() {
-    var done = this.async();
-    require('karma').server.start(this.options(), function(code) {
-      done(code === 0);
-    });
-  });
+  // Default task.
+  grunt.registerTask('default', ['build', 'connect', 'watch']);
 
 };

README.md

@@ -10,7 +10,8 @@ Proudly brought to you by the [@revolunet](http://twitter.com/revolunet) team.
 
 ## Usage :
 
- 1. Add `angular-carousel.css`, `angular-carousel.js` and `angular-mobile.js` (from this repo) to your code:
+ 1. If you use bower, just `bower install angular-carousel`.
+ 2. If not, add `angular-carousel.css`, `angular-carousel.js` to your code:
 ```html
 <link href="lib/angular-carousel.css" rel="stylesheet" type="text/css" />
 <script src="lib/angular.js"></script>
@@ -39,46 +40,26 @@ angular.module('MyApp', ['angular-carousel']);
   <li>slide #3</li>
 </ul>
 ```
- 5. Alternatively, for an infinite carousel, use the `rn-carousel-prev` and `rn-carousel-next` callbacks :
-```html
-<div rn-carousel-infinite rn-carousel-next="next(item)" rn-carousel-prev="prev(item)" rn-carousel-current="product">
-  <h1> #{{ product.id }} </h1>
-  {{ product.description }}
-</div>
-```
-
-The `prev()` and `next()` function return promises containing the prev and next slide.
 
 ## Features :
  - Mobile friendly, tested on webkit+firefox
  - CSS 3D transformations with GPU accel
+ - DOM buffering
+ - index data-binding
 
 ### Regular carousel :
  - `rn-carousel-index` two way binding to control the carousel position.
  - `rn-carousel-indicator` to turn on the indicator, see demo page.
  - `rn-carousel-buffered` to buffer the carousel, good to minimize the DOM.
- - ~~`rn-carousel-cycle` to have an forever-cycling carousel.~~ (BROKEN)
- - `rn-carousel-watch` force deep watch of the ngRepeat collection (listen to add/remove items).
-
-
-### Infinite carousel :
-
- You can setup a dynamic, infinite carousel that will load slides on demand using a promise.
- - `rn-carousel-infinite` : use this to setup an infinite carousel without the initial ul/li structure.
- - `rn-carousel-next="getNextSlide(item)"` : callback called when carousel reach the last slide, that should return a single slide. great for generating slides on-demand.
- - `rn-carousel-prev="getPrevSlide(item)"` : callback called when carousel reach the first slide, that should return a single slide. great for generating slides on-demand.
- - `rn-carousel-current` : data-binding to the current carousel item. will be sent as first argument to the prev/next callbacks.
 
 ## Todo :
- - memory profiling
- - optional auto-slide
- - buffering : allow buffer size tuning (default=3 slides)
- - buffering : add intelligent indicators
+ - see the [TODO file](./TODO)
 
 ## Inspirations
  - https://github.com/ajoslin/angular-mobile-nav
  - http://mobile.smashingmagazine.com/2012/06/21/play-with-hardware-accelerated-css/
- - Thanks @ganarajpr @bennadel and angular folks for all the tips :)
+ - http://ariya.ofilabs.com/2013/08/javascript-kinetic-scrolling-part-1.html
+ - Thanks to all angular folks for all the tips :)
 
 ## Licence
 As AngularJS itself, this module is released under the permissive [MIT license](http://revolunet.mit-license.org). Your contributions are always welcome.

TODO

@@ -0,0 +1,34 @@
+
+
+Carousel v2:
+    Base features:
+        ✔ basic ngRepeat @done (14-01-04 20:22)
+        ✔ ngRepeat with filter @done (14-01-04 20:54)
+        ✔ ngRepeat with filter and track by @done (14-01-04 20:59)
+        ✔ template based only @done (14-01-04 20:22)
+    Bindings:
+        ✔ should have a carousel-index binding @done (14-01-04 20:22)
+    Buffering:
+        ✔ should optionnaly buffer the carousel DOM @done (14-01-04 20:22)
+    Indicators:
+        ☐ should have optional indicators
+        ☐ indicators should be clickable
+        ☐ indicators should be customisable
+    Animations:
+        ☐ should have some builtin animations
+        ☐ should be able to customise animations
+    Bugs:
+        ☐ bug when rn-carousel-index change while animation from outside
+        ☐ handle touch cancel
+    Advanced:
+        ☐ optional cycle
+        ☐ infinite carousels with slide generators
+        ☐ optional auto-slide
+    Tests:
+        ✔ setup karma @done (14-01-05 00:15)
+    Tooling:
+        ✔ update grunt @done (14-01-04 22:09)
+        ✔ add autoprefixer @done (14-01-04 22:09)
+        ☐ template with code samples for the demo page
+    Docs:
+        ☐ README update

bower.json

@@ -2,15 +2,23 @@
   "author": "revolunet",
   "name": "angular-carousel",
   "description": "Mobile friendly AngularJS carousel",
-  "version": "0.0.8",
+  "version": "0.1.0",
   "homepage": "https://github.com/revolunet/angular-carousel",
   "repository": {
     "type": "git",
     "url": "git://github.com/revolunet/angular-carousel.git"
   },
-  "main": "./dist/angular-carousel.js",
+  "main": [
+    "./dist/angular-carousel.js",
+    "./dist/angular-carousel.css"
+  ],
   "dependencies": {
-    "angular": "PatternConsulting/bower-angular", 
-    "jquery": "1.9.1"
+    "angular": "1.2.7",
+    "angular-touch": "1.2.7"
+  },
+  "devDependencies": {
+    "angular": "1.2.7",
+    "angular-touch": "1.2.7",
+    "angular-mocks": "1.2.7"
   }
 }