Merge pull request #44 from karx1/dev

#43 - FIX: Optimize resources script

Author: arthtyagi

.gitignore

@@ -7,3 +7,5 @@ media/media/
 .idea/
 venv/
 __pycache__
+.vscode/
+jsconfig.json

notes/static/js/resources.js

@@ -3,7 +3,8 @@ https://github.com/The-Domecode/domecode-opensource/issues/3#issuecomment-685106
 for more info. */
 
 
-// polyfill "String.includes" function. required for bowsers without ES6 support, like Internet Explorer.
+// polyfill "String.includes" function. 
+// Required for bowsers without ES6 support, like Internet Explorer.
 if (!String.prototype.includes) {
     String.prototype.includes = function(search, start) {
         'use strict';
@@ -19,40 +20,21 @@ if (!String.prototype.includes) {
     };
 }
 
-var all = document.getElementsByTagName("*");
+// Polyfill HTMLCollection.forEach from Array.forEach
+if (!HTMLCollection.prototype.forEach) HTMLCollection.prototype.forEach = Array.prototype.forEach;
 
-var name;
-var elements = [];
+// Gather all video frames
+const frames = document.getElementsByTagName("IFRAME");
 
-// loop through all elements on page to find all the iframes
-for (var i=0, max=all.length; i < max; i++) {
-    // get element name
-    var el = all[i];
-    var name = el.nodeName.toLowerCase();
-    if (name == "iframe") {
-        // if it's an iframe, push it to the list.
-        elements.push(el);
-    }
-}
-
-// loop through all the iframes we found
-for (var i=0, max=elements.length; i < max; i++) {
-    // get the frame to work on
-    var frame = elements[i];
-    // Skip trinket python editor
-    if(frame.src.includes("trinket")) {
-        continue;
-    }
-    // Wrap the video inside the video-wrapper class
-    
-    // get the frame's current parent
-    var parent = frame.parentNode;
-    // create the wrapper div and set its class
-    var wrapper = document.createElement("div");
-    wrapper.className = "video-container"
+// Loop over each frame we got
+frames.forEach((frame) => {
+    // Get the frame's parent
+    let parent = frame.parentNode;
+    // Create the wrapper div and set its class
+    let wrapper = document.createElement("div");
+    wrapper.className = "video-container";
 
-    // set the wrapper as the parent's child instead of the frame
+    // Wrap the frame in the wrapper
     parent.replaceChild(wrapper, frame);
-    // set the frame as the child of the wrapper
     wrapper.appendChild(frame);
-}
+})