svg-sprite-loader - npm Package Compare versions

Comparing version 0.1.1 to 0.1.2

lib/web/sprite.js

@@ -101,2 +101,18 @@ var Sniffr = require('sniffr');
 /**
+ * Fix for browser (IE, maybe other too) which are throwing 'WrongDocumentError'
+ * if you insert an element which is not part of the document
+ * @see http://stackoverflow.com/questions/7981100/how-do-i-dynamically-insert-an-svg-image-into-html#7986519
+ * @param {Element} svg
+ */
+function importSvg(svg) {
+  try {
+    if (document.importNode) {
+      return document.importNode(svg, true);
+    }
+  } catch(e) {}
+
+  return svg;
+}
+
+/**
  * @type {string}
@@ -196,17 +212,9 @@ */
   var svg = new DOMParser().parseFromString(svgString, 'image/svg+xml').documentElement;
+  var importedSvg = importSvg(svg);
 
-  /**
-   * Fix for browser (IE, maybe other too) which are throwing 'WrongDocumentError'
-   * if you insert an element which is not part of the document
-   * @see http://stackoverflow.com/questions/7981100/how-do-i-dynamically-insert-an-svg-image-into-html#7986519
-   */
-  if (document.importNode) {
-    svg = document.importNode(svg, true);
-  }
-
   if (this.browser.name !== 'ie' && this.urlPrefix) {
-    baseUrlWorkAround(svg, DEFAULT_URI_PREFIX, this.urlPrefix);
+    baseUrlWorkAround(importedSvg, DEFAULT_URI_PREFIX, this.urlPrefix);
   }
 
-  return svg;
+  return importedSvg;
 };
@@ -213,0 +221,0 @@

package.json

 {
   "name": "svg-sprite-loader",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "SVG sprite webpack loader",
@@ -5,0 +5,0 @@ "main": "index.js",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

23985

increased by0.59%

Lines of code

550

increased by1.29%

No dependency changes