use copyWithin instead of loop

HansAcker · Jan 9, 2025 · cdf7cc1 · cdf7cc1
1 parent 8703295
commit cdf7cc1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/html/js/statsbox.js b/html/js/statsbox.js
@@ -96,11 +96,9 @@ class SortedStatsBox extends StatsBox {
 				this._rows[idxNew].tr.before(stat.tr); // move table row
 
 				// shift array back by one
-				while (idxOld > idxNew) {
-					this._rows[idxOld] = this._rows[idxOld-1];
-					idxOld--;
-				}
+				this._rows.copyWithin(idxNew+1, idxNew, idxOld);
 
+				// re-insert element
 				this._rows[idxNew] = stat;
 
 				//console.log(`${key}: ${this._stats[key]} => ${idxNew}`);

diff --git a/html/js/statsbox.min.js b/html/js/statsbox.min.js