Review feedback

Author: mnocon

docs/js/custom.js

@@ -80,18 +80,19 @@ $(document).ready(function() {
             const versionList = document.querySelector('.switcher__list dl.versions');
             const olderVersions = document.querySelector('#older-versions');
 
+            const versionName = (versionNode) => versionNode.textContent.trim();
+            const allVersions = [...versionList.querySelectorAll('dd')];
+
             // Remove latest version entry from the list
-            const latestVersion = [...versionList.querySelectorAll('dd')].find(v => v.textContent.trim() === 'latest');
+            const latestVersion = allVersions.find((versionNode) => versionName(versionNode) === 'latest');
             latestVersion?.remove();
 
-            const allVersions = [...versionList.querySelectorAll('dd')];
-            const versionName = (versionNode) => versionNode.textContent.trim();
-
             // Order: SaaS first, then supported versions, then EOL versions at the bottom
             const saasVersions = allVersions.filter((versionNode) => versionName(versionNode) === 'saas');
             const eolVersionNodes = allVersions.filter((versionNode) => eolVersions.includes(versionName(versionNode)));
-            const supportedVersions = allVersions.filter((versionNode) => !saasVersions.includes(versionNode) && !eolVersionNodes.includes(versionNode));
-            [...saasVersions, ...supportedVersions, ...eolVersionNodes].forEach((versionNode) => versionList.appendChild(versionNode));
+            const supportedVersions = allVersions.filter((versionNode) => versionNode !== latestVersion && !saasVersions.includes(versionNode) && !eolVersionNodes.includes(versionNode));
+            const orderedVersions = [...saasVersions, ...supportedVersions, ...eolVersionNodes];
+            orderedVersions.forEach((versionNode) => versionList.appendChild(versionNode));
 
             if (eolVersionNodes.length > 0) {
                 olderVersions.hidden = false;
@@ -101,14 +102,13 @@ $(document).ready(function() {
                 versionNode.hidden = true;
             });
 
-            allVersions
-                .forEach((versionNode) => {
-                    versionNode.querySelector('a').textContent = versionLabels[versionName(versionNode)] ?? versionName(versionNode);
-                });
+            orderedVersions.forEach((versionNode) => {
+                versionNode.querySelector('a').textContent = versionLabels[versionName(versionNode)] ?? versionName(versionNode);
+            });
 
             olderVersions.addEventListener('click', (event) => {
                 event.stopPropagation();
-                allVersions.forEach((versionNode) => {
+                orderedVersions.forEach((versionNode) => {
                     versionNode.hidden = false;
                 });
                 olderVersions.hidden = true;