Skip to content

[lexical-playground] Bug Fix: processing html in paste command to correct markup #7711

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

harshmetkel24
Copy link
Contributor

Description

Describe the changes in this pull request
processing markup exported when content is copied from Microsoft Office to lexical to correct semantics

Closes #4761

Test plan

Before

Screen.Recording.2025-07-21.at.4.10.57.PM.mov

Insert relevant screenshots/recordings/automated-tests

After

Insert relevant screenshots/recordings/automated-tests

Screen.Recording.2025-07-21.at.4.11.47.PM.mov

Copy link

vercel bot commented Jul 21, 2025

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 Jul 22, 2025 9:58am
lexical-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 22, 2025 9:58am

@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 Jul 21, 2025
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.

Instead of integrating this directly into the Editor it would make sense to have it be a separate plug-in component. It looks like there are also lint/type issues that need to be addressed

@harshmetkel24
Copy link
Contributor Author

Agree @etrepum my initial intention were to get started with whether my approach is correct or not. Will surely update code to move logic to separate plugin along with lint fixes
Thanks 🙌🏾

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 like type check is failing, npm run ci-check will run all of the integrity tests

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.

Feature: Copy Paste Bullet/Number list from MS Word Document
2 participants