Merge pull request #816 from hargata/Hargata/refine.sort

only add the default sorter once.
hargata · Jan 21, 2025 · f1f99a6 · f1f99a6
2 parents c11dad0 + a4f15ff
commit f1f99a6
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 7 deletions.
diff --git a/wwwroot/js/garage.js b/wwwroot/js/garage.js
@@ -279,9 +279,11 @@ function sortGarage(sender, isMobile) {
             sender.addClass('sort-asc');
             sender.html(isMobile ? `<span class="ms-2 display-3">${garageIcon}${sortColumn}${sortAscIcon}</span>` : `${garageIcon}${sortColumn}${sortAscIcon}`);
             //append sortRowId to the vehicle container
-            $(`.garage-item`).map((index, elem) => {
-                $(elem).attr("default-sort", index);
-            });
+            if ($("[default-sort]").length == 0) {
+                $(`.garage-item`).map((index, elem) => {
+                    $(elem).attr("default-sort", index);
+                });
+            }
             sortVehicles(false);
         }
     }

diff --git a/wwwroot/js/shared.js b/wwwroot/js/shared.js
@@ -446,10 +446,12 @@ function toggleSort(tabName, sender) {
         }
         sender.addClass('sort-asc');
         sender.html(`${sortColumn}${sortAscIcon}`);
-        //append sortRowId to the table rows.
-        $(`#${tabName} table tbody tr`).map((index, elem) => {
-            $(elem).attr("default-sort", index);
-        });
+        //append sortRowId to the table rows if nothing has been appended yet.
+        if ($("[default-sort]").length == 0) {
+            $(`#${tabName} table tbody tr`).map((index, elem) => {
+                $(elem).attr("default-sort", index);
+            });
+        }
         sortTable(tabName, sortColumn, false);
     }
 }