Skip to content

Bulk edit: Insert columns in initial order after filtering [MAPS-41]#10139

Merged
Joaquin (joaquincasal) merged 1 commit intomasterfrom
maps-51-column-order
Sep 30, 2025
Merged

Bulk edit: Insert columns in initial order after filtering [MAPS-41]#10139
Joaquin (joaquincasal) merged 1 commit intomasterfrom
maps-51-column-order

Conversation

@joaquincasal
Copy link
Copy Markdown
Collaborator

Purpose

Right now if you deselect a column and add it again, it will be added at the end of the table instead of its original position

Approach

Every time we insert a column into the selectedColumns state, we reorder it

Testing steps

Remove and add a column from the columns filter

Screen.Recording.2025-09-30.at.10.08.08.mov

@joaquincasal Joaquin (joaquincasal) requested a review from a team as a code owner September 30, 2025 13:11
Copy link
Copy Markdown
Collaborator

@FBanfi Franco Banfi (FBanfi) left a comment

Choose a reason for hiding this comment

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

🚀

Copy link
Copy Markdown
Contributor

@ryunsong-contentful ryunsong-contentful left a comment

Choose a reason for hiding this comment

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

Looking at this PR to review the column ordering implementation...

Copy link
Copy Markdown
Contributor

@ryunsong-contentful ryunsong-contentful left a comment

Choose a reason for hiding this comment

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

Nice

@joaquincasal Joaquin (joaquincasal) merged commit 33b1dee into master Sep 30, 2025
15 checks passed
@joaquincasal Joaquin (joaquincasal) deleted the maps-51-column-order branch September 30, 2025 16:17
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.

3 participants