Merge pull request #87 from joetuson/film-options

respect the film-strip option

README.md

@@ -364,6 +364,27 @@ grunt.initConfig( {
 } );
 ```
 
+### Troubleshooting
+
+By default, the experimental `film-strip` option is true. If the grunt phantomas command fails, try setting the film strip option to false:
+
+```js
+grunt.initConfig( {
+  phantomas: {
+    yourSite : {
+      options : {
+        indexPath : './yoursite/',
+        options   : {
+          'film-strip'   : false
+        },
+        url       : 'http://yoursite.com'
+      }
+    }
+  }
+} );
+```
+
+
 ### Tracking history in CI
 
 To track history in Travis CI, use the [caching option](http://docs.travis-ci.com/user/caching/#Arbitrary-directories) to cache the `indexPath` folder.

tasks/lib/phantomas.js

@@ -308,7 +308,7 @@ Phantomas.prototype.executePhantomas = function() {
       options = _.clone( this.options.options );
 
       // run it only for the first run
-      if ( i === 0 ) {
+      if ( i === 0 && options[ 'film-strip' ] !== false ) {
         options[ 'film-strip' ]     = true;
         options[ 'film-strip-dir' ] = this.imagePath + this.timestamp;
       }

tasks/tpl/index.tpl

@@ -58,8 +58,8 @@
           </ul>
         <% } %>
       <% } %>
-      <h2>Last run film strip for <%= url %></h2>
       <% if ( images.length ) { %>
+        <h2>Last run film strip for <%= url %></h2>
         <ul class="p--filmstrip">
           <% _.each( images, function( image ) { %>
             <li><img src="images/<%= timestamp %>/<%= image %>" alt="film strip image of <%= url %>">