Skip to content

Remove a special case in table deleteTextHandler#5891

Merged
ivailop7 merged 2 commits intofacebook:mainfrom
AlexanderReznik:table-delete-character
Apr 16, 2024
Merged

Remove a special case in table deleteTextHandler#5891
ivailop7 merged 2 commits intofacebook:mainfrom
AlexanderReznik:table-delete-character

Conversation

@AlexanderReznik
Copy link
Copy Markdown
Contributor

This was introduced in #3213 but wasn't cleaned up after a proper fix.
Currently this does not allow the removal of empty lines in a collapsible section inside of a table.

Before:
table-collapsible-paragraph

After:
table-collapsible-paragraph-after

@facebook-github-bot facebook-github-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 Apr 15, 2024
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
lexical ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 16, 2024 9:39am
lexical-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 16, 2024 9:39am

@StyleT
Copy link
Copy Markdown
Contributor

StyleT commented Apr 15, 2024

Hi! Thanks for this fix! Do you think we can write a test for it?

zurfyx
zurfyx previously approved these changes Apr 15, 2024
@AlexanderReznik
Copy link
Copy Markdown
Contributor Author

Hi! Thanks for this fix! Do you think we can write a test for it?

Sure! d95d99f

`,
);
});

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@ivailop7 ivailop7 merged commit 3ae807a into facebook:main Apr 16, 2024
@AlexanderReznik AlexanderReznik deleted the table-delete-character branch April 16, 2024 13:01
@acywatson acywatson mentioned this pull request Apr 16, 2024
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants