(superset) branch amaan/fix-ag-grid-table-summary-row created (now 9eb5d2ee5ab)

[am...@apache.org]

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

amaannawab923 pushed a change to branch amaan/fix-ag-grid-table-summary-row
in repository https://gitbox.apache.org/repos/asf/superset.git


      at 9eb5d2ee5ab fix(plugin-chart-ag-grid-table): render Show summary totals row on ag-grid 36

This branch includes the following new commits:

     new 9eb5d2ee5ab fix(plugin-chart-ag-grid-table): render Show summary totals row on ag-grid 36

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

(superset) 01/01: fix(plugin-chart-ag-grid-table): render Show summary totals row on ag-grid 36

[am...@apache.org]

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

amaannawab923 pushed a commit to branch amaan/fix-ag-grid-table-summary-row
in repository https://gitbox.apache.org/repos/asf/superset.git

commit 9eb5d2ee5ab2713dabbd6a2a5374298f1205e323
Author: Amaan <am...@gmail.com>
AuthorDate: Fri Jul 17 01:13:01 2026 +0530

    fix(plugin-chart-ag-grid-table): render Show summary totals row on ag-grid 36
    
    The "Show summary" totals row is invisible on the AG Grid Interactive
    Table after the ag-grid 36 upgrade (#41882). The pinned bottom row is
    built and populated correctly (the DOM contains the row with the correct
    aggregated values), but ag-grid 36 leaves its row-entrance animation
    class `ag-opacity-zero` (opacity: 0) permanently applied to the pinned
    bottom row. Normally ag-grid removes that class on the next frame to fade
    the row in; under v36 it is never removed for pinned rows, so the summary
    row renders fully but stays transparent.
    
    Force pinned rows opaque in the chart's styled container, overriding the
    stuck ag-grid animation class, so the summary row is visible again.
---
 .../plugins/plugin-chart-ag-grid-table/src/styles/index.tsx  | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/superset-frontend/plugins/plugin-chart-ag-grid-table/src/styles/index.tsx b/superset-frontend/plugins/plugin-chart-ag-grid-table/src/styles/index.tsx
index 7df56939218..ed61890e710 100644
--- a/superset-frontend/plugins/plugin-chart-ag-grid-table/src/styles/index.tsx
+++ b/superset-frontend/plugins/plugin-chart-ag-grid-table/src/styles/index.tsx
@@ -361,6 +361,18 @@ export const StyledChartContainer = styled.div<{
       font-size: ${theme.fontSizeSM}px;
     }
 
+    /*
+     * AG Grid 34+ adds the row-entrance-animation class 'ag-opacity-zero'
+     * (opacity: 0) to a newly inserted row and removes it on the next frame to
+     * fade the row in. Under AG Grid 36 that class is never removed from the
+     * pinned bottom row, so the "Show summary" totals row renders fully (correct
+     * values in the DOM) but stays permanently transparent. Force pinned rows
+     * opaque so the summary row is visible.
+     */
+    .ag-row-pinned {
+      opacity: 1 !important;
+    }
+
     .ag-spanned-row {
       font-size: ${theme.fontSizeSM}px;
       font-weight: ${theme.fontWeightStrong};