Skip to content

fix(table): resize grouped header columns correctly - #1394

Open
xonaib wants to merge 1 commit into
openng-org:mainfrom
xonaib:fix-table-resize-grouped-header
Open

fix(table): resize grouped header columns correctly#1394
xonaib wants to merge 1 commit into
openng-org:mainfrom
xonaib:fix-table-resize-grouped-header

Conversation

@xonaib

@xonaib xonaib commented Jul 24, 2026

Copy link
Copy Markdown

Summary

  • measure the active resize header row instead of every header cell in the table head
  • keep a fallback for tables without an active resize column
  • add regression coverage for resizable columns under grouped headers

Fixes #78

Test plan

  • pnpm --filter @openng/optimus-ui test:unit --include=src/table/table.spec.ts
  • pnpm run build:lib

@xonaib

xonaib commented Jul 24, 2026

Copy link
Copy Markdown
Author

I saw there was an older PR/comment around this issue, but that PR appears to have been closed as stale/too old, with a note that a fresh PR would be welcome if the issue is still relevant. This is a fresh pass against the current codebase and includes regression coverage for grouped headers.

Tested locally:

  • pnpm --filter @openng/optimus-ui test:unit --include=src/table/table.spec.ts
  • pnpm run build:lib

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.

resizableColumns broken when using a second header row (column grouping) in p-table

1 participant