Add comments

Author: kgryte

spec/_static/javascripts/version_dropdown.js

@@ -6,12 +6,12 @@
 * @param {string} path - resource path
 * @returns {Promise} promise which resolves a resource URL
 */
-async function href(url, path) {
+function href(url, path) {
     const defaultURL = url + "/index.html";
     url += "/" + path;
 
     // If a versioned resource exists, return the resource's URL; otherwise, return a default URL:
-    await fetch(url).then(onResponse).catch(onError);
+    return fetch(url).then(onResponse).catch(onError);
 
     /**
     * Callback invoked upon successfully resolving a resource.
@@ -68,25 +68,44 @@ async function add_version_dropdown(json_loc, target_loc, text) {
     * @returns {Promise} promise which resolves upon processing version data
     */
     async function onDone(versions) {
-        console.log(versions)
+        console.log(versions);
+        
+        // Resolve the current browser URL:
         const currentURL = window.location.href;
+
+        // Check whether the user is currently on a resource page (e.g., is viewing the specification for a particular function):
         let path = currentURL.split(/_site|array_api/)[1];
+
+        // Extract the resource subpath:
         if (path) {
             path = path.split("/");
             path = path.slice(2, path.length);
             path = path.join("/");
         } else {
             path = "";
         }
+        // For each version, create an anchor element and attempt to resolve a given resource for that version...
+        const promises = [];
+        const el = [];
         for (let key in versions) {
             if (versions.hasOwnProperty(key)) {
                 let a = document.createElement("a");
                 a.innerHTML = key;
                 a.title = key;
-                a.href = await href(target_loc + versions[key], path);
-                content.appendChild(a);
+                el.push(a);
+                promises.push(href(target_loc + versions[key], path));
             }
         }
+        // Resolve all resource URLs:
+        const urls = await Promise.all(promises);
+
+        // Append the version links to the dropdown menu:
+        for (let i = 0; i < urls.length; i++) {
+            let a = el[i];
+            a.href = urls[i];
+            content.appendChild(a);
+        }
+        // Set the button text:
         button.innerHTML = text;
     }