From ed78f56e6b290e835846bb4c4ff415daea9331bb Mon Sep 17 00:00:00 2001
From: William Allen <william.allen@kitware.com>
Date: Wed, 28 Jan 2026 12:54:21 -0500
Subject: [PATCH] Fix coverage sort order

The line and branch percentages were introduced in #3348 as text columns instead of numeric columns.  This PR fixes the issue by casting the value to a float, allowing the values to be sorted numerically instead of alphabetically.
---
 .../js/vue/components/BuildCoveragePage.vue   | 27 ++++++++++++++-----
 1 file changed, 21 insertions(+), 6 deletions(-)

diff --git a/resources/js/vue/components/BuildCoveragePage.vue b/resources/js/vue/components/BuildCoveragePage.vue
index 55a863d88d..4a69dbb34e 100644
--- a/resources/js/vue/components/BuildCoveragePage.vue
+++ b/resources/js/vue/components/BuildCoveragePage.vue
@@ -150,7 +150,7 @@
               <font-awesome-icon :icon="FA.faFile" /> {{ obj.path }}
             </a>
           </template>
-          <template #linePercentage="{ props: pct }">
+          <template #linePercentage="{ props: { text: pct } }">
             <progress
               class="tw-progress tw-w-24"
               :class="percentToProgressBarColorClass(pct)"
@@ -158,7 +158,7 @@
               max="100"
             /> {{ pct }}%
           </template>
-          <template #branchPercentage="{ props: pct }">
+          <template #branchPercentage="{ props: { text: pct } }">
             <progress
               class="tw-progress tw-w-24"
               :class="percentToProgressBarColorClass(pct)"
@@ -384,12 +384,27 @@ export default {
       });
 
       return Object.values(coverageByPrefix).map(obj => {
+        const linePct = this.computePercentage(obj.linesOfCodeTested, obj.linesOfCodeUntested);
+        const branchPct = this.computePercentage(obj.branchesTested, obj.branchesUntested);
+
         return {
           ...obj,
-          linePercentage: this.computePercentage(obj.linesOfCodeTested, obj.linesOfCodeUntested),
-          lines: `${obj.linesOfCodeTested} / ${obj.linesOfCodeTested + obj.linesOfCodeUntested}`,
-          branchPercentage: this.computePercentage(obj.branchesTested, obj.branchesUntested),
-          branches: `${obj.branchesTested} / ${obj.branchesTested + obj.branchesUntested}`,
+          linePercentage: {
+            text: linePct,
+            value: parseFloat(linePct),
+          },
+          lines: {
+            text: `${obj.linesOfCodeTested} / ${obj.linesOfCodeTested + obj.linesOfCodeUntested}`,
+            value: parseInt(obj.linesOfCodeTested),
+          },
+          branchPercentage: {
+            text: branchPct,
+            value: parseFloat(branchPct),
+          },
+          branches: {
+            text: `${obj.branchesTested} / ${obj.branchesTested + obj.branchesUntested}`,
+            value: parseInt(obj.branchesTested),
+          },
         };
       });
     },