Skip to content

[lexical-markdown] Bug Fix: Fix implicit checklist marker export regression#7928

Merged
etrepum merged 2 commits intofacebook:mainfrom
etrepum:markdown-shortcut-checkbox-marker
Oct 17, 2025
Merged

[lexical-markdown] Bug Fix: Fix implicit checklist marker export regression#7928
etrepum merged 2 commits intofacebook:mainfrom
etrepum:markdown-shortcut-checkbox-marker

Conversation

@etrepum
Copy link
Copy Markdown
Collaborator

@etrepum etrepum commented Oct 17, 2025

Description

When a markdown shortcut creates a checklist without an explicit list marker it would remember the [ instead of a valid list marker (+, -, or *). This addresses that regression from #7892 and ensures that invalid markers won't be parsed from JSON serialization.

Closes #7927

Test plan

Unit tests

@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 Oct 17, 2025
@vercel
Copy link
Copy Markdown

vercel bot commented Oct 17, 2025

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

Project Deployment Preview Comments Updated (UTC)
lexical Ready Ready Preview Comment Oct 17, 2025 6:12pm
lexical-playground Ready Ready Preview Comment Oct 17, 2025 6:12pm

💡 Enable Vercel Agent with $100 free credit for automated AI reviews

@etrepum etrepum added this pull request to the merge queue Oct 17, 2025
Merged via the queue into facebook:main with commit 0d6ff68 Oct 17, 2025
39 checks passed
This was referenced Oct 27, 2025
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.

Bug: Checklist markdown not exported correctly

2 participants