(superset) branch master updated: fix(table): ensure dimensions appear before metrics in column order (#39346)

[ma...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

mistercrunch pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 4f19bc4c5f7 fix(table): ensure dimensions appear before metrics in column order (#39346)
4f19bc4c5f7 is described below

commit 4f19bc4c5f7af6b603c8634442130777e31f1ec6
Author: Maxime Beauchemin <ma...@gmail.com>
AuthorDate: Mon Apr 20 17:44:26 2026 -0700

    fix(table): ensure dimensions appear before metrics in column order (#39346)
    
    Co-authored-by: Claude Opus 4.6 <no...@anthropic.com>
---
 .../plugins/plugin-chart-ag-grid-table/src/transformProps.ts         | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/superset-frontend/plugins/plugin-chart-ag-grid-table/src/transformProps.ts b/superset-frontend/plugins/plugin-chart-ag-grid-table/src/transformProps.ts
index c27bd856d89..33ac8aa0183 100644
--- a/superset-frontend/plugins/plugin-chart-ag-grid-table/src/transformProps.ts
+++ b/superset-frontend/plugins/plugin-chart-ag-grid-table/src/transformProps.ts
@@ -431,6 +431,11 @@ const processColumns = memoizeOne(function processColumns(
         formatter,
         config,
       };
+    })
+    .sort((a, b) => {
+      const aIsMetric = a.isMetric || a.isPercentMetric ? 1 : 0;
+      const bIsMetric = b.isMetric || b.isPercentMetric ? 1 : 0;
+      return aIsMetric - bIsMetric;
     });
   return [metrics, percentMetrics, columns] as [
     typeof metrics,