Skip to content

fix: DH-20415: AG Grid headers should match column names#1261

Merged
mofojed merged 6 commits intodeephaven:mainfrom
mofojed:DH-20415-ag-grid-header-name
Nov 13, 2025
Merged

fix: DH-20415: AG Grid headers should match column names#1261
mofojed merged 6 commits intodeephaven:mainfrom
mofojed:DH-20415-ag-grid-header-name

Conversation

@mofojed
Copy link
Member

@mofojed mofojed commented Nov 10, 2025

  • Previously we were omitting the headerName property of the column definition. When omitted, AG Grid automatically reformats the header name and adds spaces if it is a PascalCase header name
  • Explicitly define headerName in the ColumnDefinition to match the actual name of the column
  • Added unit tests, updated e2e tests

- Previously we were omitting the `headerName` property of the column definition. When omitted, AG Grid automatically reformats the header name and adds spaces if it is a PascalCase header name
- Explicitly define `headerName` in the ColumnDefinition to match the actual name of the column
@mofojed mofojed requested a review from mattrunyon November 10, 2025 21:06
@mofojed mofojed self-assigned this Nov 10, 2025
@mofojed mofojed force-pushed the DH-20415-ag-grid-header-name branch from 6b0a0e1 to b34830a Compare November 10, 2025 21:06
@mofojed mofojed requested a review from mattrunyon November 12, 2025 14:29
mattrunyon
mattrunyon previously approved these changes Nov 13, 2025
Copy link
Collaborator

@mattrunyon mattrunyon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like there's an e2e snapshot merge conflict

@mofojed mofojed merged commit c58ccec into deephaven:main Nov 13, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants