Skip to content

[lexical-playground]: Rearrange table columns#8063

Merged
ivailop7 merged 16 commits intofacebook:mainfrom
ivailop7:column_rearrange
Feb 10, 2026
Merged

[lexical-playground]: Rearrange table columns#8063
ivailop7 merged 16 commits intofacebook:mainfrom
ivailop7:column_rearrange

Conversation

@ivailop7
Copy link
Copy Markdown
Collaborator

@ivailop7 ivailop7 commented Jan 4, 2026

Add a drag handle button to the table hover action buttons.
Leverages the atlaskit npm packages.
Also fixes a number of FloatingUI issues.
Only works if the table does not contain any merged cells.

Screen.Recording.2026-01-04.at.23.48.44.mov

@vercel
Copy link
Copy Markdown

vercel bot commented Jan 4, 2026

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

Project Deployment Actions Updated (UTC)
lexical Ready Ready Preview, Comment Feb 10, 2026 8:17pm
lexical-playground Ready Ready Preview, Comment Feb 10, 2026 8:17pm

Request Review

@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 Jan 4, 2026
@ivailop7 ivailop7 added the extended-tests Run extended e2e tests on a PR label Jan 4, 2026
@etrepum
Copy link
Copy Markdown
Collaborator

etrepum commented Jan 5, 2026

Looks like this one needs new test expectations

ivailop7 and others added 5 commits February 8, 2026 22:39
The column rearrangement feature adds data-drop-target-for-element
attributes to header cells at runtime, which is irrelevant to content
assertions and was causing 7 table E2E test failures.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ivailop7
Copy link
Copy Markdown
Collaborator Author

ivailop7 commented Feb 9, 2026

@etrepum this one is ready for rereview

Copy link
Copy Markdown
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.

I didn't take a super close look at the code this time but it worked great when I tried it in the playground!

package.json Outdated
Comment on lines +212 to +213
"@atlaskit/drag-and-drop-indicator": "^0.11.1",
"@atlaskit/pragmatic-drag-and-drop": "^1.7.7",
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think these should go into packages/lexical-playground/package.json

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

yep, fixing this.

@ivailop7 ivailop7 added this pull request to the merge queue Feb 10, 2026
Merged via the queue into facebook:main with commit 3f56303 Feb 10, 2026
34 of 36 checks passed
@etrepum etrepum mentioned this pull request Feb 25, 2026
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