Skip to content

Conversation

@ivailop7
Copy link
Collaborator

@ivailop7 ivailop7 commented Dec 30, 2025

Add basic sorting to table columns.

Behaviour:
On mouse hover over the top row, it shows a sort button that shows a menu for Ascending or Descending sorting of the column. Upon selection it sorts the rows in the table.It excludes the header row from the sort if such is present.
The sorting is no-op when the table contains any merged cells.

Screen.Recording.2025-12-30.at.22.37.23.mov

@vercel
Copy link

vercel bot commented Dec 30, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
lexical Ready Ready Preview, Comment Jan 2, 2026 8:17pm
lexical-playground Ready Ready Preview, Comment Jan 2, 2026 8:17pm

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 30, 2025
@ivailop7 ivailop7 marked this pull request as ready for review December 30, 2025 20:57
@ivailop7 ivailop7 added the extended-tests Run extended e2e tests on a PR label Dec 30, 2025
@ivailop7 ivailop7 changed the title [lexical-playground]: Table Column Sort [lexical-playground]: Column Sort for Basic Table Dec 30, 2025
@ivailop7
Copy link
Collaborator Author

@etrepum thanks for the pointers! Addressed the comments and made it no-op when there are merged cells. I think is in a mergable state now.

Copy link
Collaborator

@etrepum etrepum left a comment

Choose a reason for hiding this comment

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

Looks and works great!

@etrepum etrepum added this pull request to the merge queue Jan 2, 2026
Merged via the queue into facebook:main with commit 5bbbe84 Jan 2, 2026
38 of 39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. extended-tests Run extended e2e tests on a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants