merge 'feat: order traditions alphabetically'

tla · Feb 23, 2024 · 64113f2 · 64113f2
2 parents 9a7ea1e + 0ac8d61
commit 64113f2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/frontend/www/src/js/modules/dashboard/tradition/editStemma.js b/frontend/www/src/js/modules/dashboard/tradition/editStemma.js
@@ -27,6 +27,7 @@ class EditStemma extends HTMLElement {
         // unless last re-render is active or only very shortly finished (TODO: How?)
         stemmaRenderer.render_stemma( stemmaRenderer.ellipse_border_to_none( editor_dot ) );
       } catch( { name, message } ) {
+        // console.log( name, message );
         // Nothing happens when we do not have a well formed dot string.
       }
     });

diff --git a/frontend/www/src/js/modules/dashboard/tradition/traditionList.js b/frontend/www/src/js/modules/dashboard/tradition/traditionList.js
@@ -100,13 +100,18 @@ class TraditionList extends HTMLElement {
         return traditionListItem;
     }
 
+    alphabetizeTraditions( traditions ) {
+        return traditions.sort( (tradition_a, tradition_b) => tradition_a.name.localeCompare( tradition_b.name ) );
+    }
+
     /**
      * Renders a container for the traditions and sections navigation tree.
      * Adds a list item for each tradition using @see SectionPropertiesView.createTraditionListItem.
      * 
      * @param {Tradition[]} traditions 
      */
     render( traditions ) {
+        const orderedTraditions = this.alphabetizeTraditions( traditions );
         this.innerHTML = `
             <ul id="traditions-list" class="nav flex-column mb-2"></ul>`;
         const traditionListElement = this.querySelector( 'ul' )