0.0.3

Author: blai

index.js

@@ -222,26 +222,32 @@ var spriteSVG = function(options) {
         if (file.isStream()) {
             return cb(new gutil.PluginError(PLUGIN_NAME, 'Streams are not supported'));
         }
+        console.log(file.path);
             // We're using the filename as the CSS class name
         var filename = path.basename(file.relative, path.extname(file.relative)),
             // Load the file contents
             $file = cheerio.load(file.contents.toString('utf8'), {xmlMode: true})('svg'),
-            viewBox = $file.attr('viewBox'),
-            coords = viewBox.split(" ");
-
-        // Set sprite data to be used by the positioning function
-        var sprite = {
-                fileName: filename,
-                file: $file.contents(),
-                h: parseFloat(coords[3]),
-                padding: options.padding,
-                // Round up coordinates to avoid chopping off edges
-                viewBox: Math.ceil(coords[0])+" "+Math.ceil(coords[1])+" "+Math.ceil(coords[2])+" "+Math.ceil(coords[3]),
-                w: parseFloat(coords[2])
-            };
-
-        // Add the sprite to our array
-        data.sprites.push(sprite);
+            viewBox = $file.attr('viewBox');
+
+        if (viewBox) {
+            var coords = viewBox.split(" ");
+
+            // Set sprite data to be used by the positioning function
+            var sprite = {
+                    fileName: filename,
+                    file: $file.contents(),
+                    h: parseFloat(coords[3]),
+                    padding: options.padding,
+                    // Round up coordinates to avoid chopping off edges
+                    viewBox: Math.ceil(coords[0])+" "+Math.ceil(coords[1])+" "+Math.ceil(coords[2])+" "+Math.ceil(coords[3]),
+                    w: parseFloat(coords[2])
+                };
+
+            // Add the sprite to our array
+            data.sprites.push(sprite);
+        } else {
+           gutil.log('Warning: svg "' + file.path + '" has no viewBox, not adding it to sprite');    
+        }
 
         // Move on to processSprites()
         cb();

package.json

@@ -1,36 +1,30 @@
 {
   "name": "gulp-svg-spritesheet",
   "description": "Gulp SVG sprite sheet generator",
-  "version": "0.0.2",
+  "version": "0.0.3",
   "author": "Darren Hall <[email protected]>",
   "license": "MIT",
-
   "repository": {
     "type": "git",
     "url": "https://github.com/iamdarrenhall/gulp-svg-spritesheet.git"
   },
-
   "homepage": "https://github.com/iamdarrenhall/gulp-svg-spritesheet",
   "bugs": {
     "url": "https://github.com/iamdarrenhall/gulp-svg-spritesheet/issues"
   },
-
   "keywords": [
     "gulpplugin",
     "sprite",
     "svg"
   ],
-
   "engines": {
     "node": ">=0.10.28"
   },
-
   "devDependencies": {
     "gulp": "^3.8.9",
     "gulp-jshint": "^1.9.0",
     "gulp-svgmin": "^0.4.7"
   },
-
   "dependencies": {
     "cheerio": "^0.17.0",
     "gulp-util": "^3.0.1",