Fix IE9 incompatible classList JS

Additionally add a check for whether the class has already been applied, so that (e.g., on page resize) the class doesn't get infinitely appended
cee-chen · Apr 24, 2017 · a5cbef3 · a5cbef3
1 parent d8a9a8f
commit a5cbef3
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 4 deletions.
diff --git a/dist/objectFitPolyfill.basic.min.js b/dist/objectFitPolyfill.basic.min.js
diff --git a/dist/objectFitPolyfill.min.js b/dist/objectFitPolyfill.min.js
diff --git a/src/objectFitPolyfill.basic.js b/src/objectFitPolyfill.basic.js
@@ -46,7 +46,10 @@
       $container.style.height = "100%";
     }
 
-    $container.classList.add("object-fit-polyfill");
+    // Add a CSS class hook, in case people need to override styles for any reason.
+    if ($container.className.indexOf("object-fit-polyfill") === -1) {
+      $container.className = $container.className + " object-fit-polyfill";
+    }
   };
 
   /**

diff --git a/src/objectFitPolyfill.js b/src/objectFitPolyfill.js
@@ -43,7 +43,10 @@
       $container.style.height = "100%";
     }
 
-    $container.classList.add("object-fit-polyfill");
+    // Add a CSS class hook, in case people need to override styles for any reason.
+    if ($container.className.indexOf("object-fit-polyfill") === -1) {
+      $container.className = $container.className + " object-fit-polyfill";
+    }
   };
 
   /**