Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[BUG] Crash when pasting nested list of richtext #498

Closed
2 tasks done
TomNUSDS opened this issue Jun 18, 2024 · 2 comments
Closed
2 tasks done

[BUG] Crash when pasting nested list of richtext #498

TomNUSDS opened this issue Jun 18, 2024 · 2 comments
Labels
bug Something isn't working duplicate This issue or pull request already exists

Comments

@TomNUSDS
Copy link

  • I have searched for similar issues in both open and closed tickets and cannot find a duplicate.
  • I have read the documentation and cannot find an answer.

Describe the bug
Pasting nested lists into the richtext editor causes an exception.

To Reproduce
Steps to reproduce the behavior:

  1. Go to MDX Editor Live demo site
  2. Clear the content in the richtext editor so it's empty
  3. Open browser debugger to watch for crash
  4. Extract attached richtext example data from test.rtf.zip
  5. Open test.rtf with any editor that supports richtext
  6. Copy content and paste into the MDXEditor richtext area.
    Result: crashes
    Uncaught TypeError: Cannot read properties of undefined (reading 'children') at h
    Note: Switching to Markdown view in MDXEditor shows empty content.

Desktop (please complete the following information):

  • OS: MacOS
  • Browser: Chrome

Additional context
The code is minimized, but I think it's in isParent() when the node parameter is null for some reason.

test.rtf.zip
@TomNUSDS TomNUSDS added the bug Something isn't working label Jun 18, 2024
@petyosi
Copy link
Contributor

petyosi commented Jun 19, 2024

This is a duplicate of #178

@petyosi petyosi closed this as not planned Won't fix, can't repro, duplicate, stale Jun 19, 2024
@petyosi petyosi added the duplicate This issue or pull request already exists label Jun 19, 2024
@TomNUSDS
Copy link
Author

Sorry for the duplicate. I did search through the issues, must of just missed it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@petyosi @TomNUSDS