Skip to content

chore: add the ability to pin columns#1781

Open
scottdover wants to merge 4 commits intomainfrom
feat/1720-column-filter
Open

chore: add the ability to pin columns#1781
scottdover wants to merge 4 commits intomainfrom
feat/1720-column-filter

Conversation

@scottdover
Copy link
Contributor

@scottdover scottdover commented Feb 6, 2026

Summary:
This does a few things to enable column pinning:

  • First, it refines the persisted column state to only store what is necessary:
    id, sort type, sort index, pinned status.
  • Second, it adds a new menu item to pin columns. Since the pin columns sub
    menu is larger than the sort menu, and since we don't unmount the child
    component before re-render, an id attribute was introduced to "force"
    a resize.
  • Third, this addresses an issue where calling applyColumnState with a partial
    column state was resulting in data loss (sort,pinned status loss). To fix that,
    all actions now update all column state instead of making partial updates.

Testing:

  • Make sure pinning columns works in conjunction with filter/sort
  • Make sure column pinning works when switching between multiple tables

Scott Dover added 2 commits February 6, 2026 13:16
Signed-off-by: Scott Dover <Scott.Dover@sas.com>
Signed-off-by: Scott Dover <Scott.Dover@sas.com>
@scottdover scottdover force-pushed the feat/1720-column-filter branch from bca5f3e to 421fb41 Compare February 6, 2026 18:16
Signed-off-by: Scott Dover <Scott.Dover@sas.com>
@scottdover scottdover marked this pull request as ready for review February 6, 2026 18:34
Signed-off-by: Scott Dover <Scott.Dover@sas.com>
@scottdover scottdover force-pushed the feat/1720-column-filter branch from a297496 to 82cd872 Compare February 6, 2026 19:19
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.

1 participant