[Ag Grid Theme] Remove added styles when autoHeaderHeight is applied (#…

…2656)
jpmorganchase · Oct 30, 2023 · d200218 · d200218 · vercel · Oct 30, 2023
1 parent 6da4d09
commit d200218
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/.changeset/curly-lies-wave.md b/.changeset/curly-lies-wave.md
@@ -0,0 +1,5 @@
+---
+"@salt-ds/ag-grid-theme": patch
+---
+
+Ag Grid Theme - remove override. set config correctly to wrap header text
diff --git a/packages/ag-grid-theme/css/_salt-ag-theme-mixin.scss b/packages/ag-grid-theme/css/_salt-ag-theme-mixin.scss
@@ -296,11 +296,6 @@
     margin-left: var(--salt-spacing-100);
   }
 
-  .ag-header-cell-auto-height .ag-header-cell-text {
-    overflow: hidden;
-    white-space: pre-wrap;
-  }
-
   .ag-keyboard-focus .ag-header-cell:focus:after {
     height: auto;
     width: auto;

diff --git a/packages/ag-grid-theme/stories/examples/WrappedHeader.tsx b/packages/ag-grid-theme/stories/examples/WrappedHeader.tsx
@@ -67,6 +67,7 @@ const WrappedHeader = (props: AgGridReactProps) => {
           defaultColDef={{
             ...propsColDefs,
             autoHeaderHeight: true,
+            wrapHeaderText: true,
           }}
           {...restAgGridProps}
           {...props}