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.

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

Undo operation malfunctions when there are line breaks #227

Closed
2 of 3 tasks
juanmagiraldor opened this issue Oct 2, 2023 · 1 comment
Closed
2 of 3 tasks

Undo operation malfunctions when there are line breaks #227

juanmagiraldor opened this issue Oct 2, 2023 · 1 comment
Labels
🐞 Bug Something isn't working ⛑ Needs triage Someone is checking

Comments

@juanmagiraldor
Copy link
Contributor

juanmagiraldor commented Oct 2, 2023

Precheck

  • Do a quick search and make sure the bug has not been reported yet.
  • Please disclose security vulnerabilities privately at oss@kommit.co.
  • Finally, be nice and have fun!

Description

In the editor, there appears to be a visual error where, when performing an undo operation with specific steps, the saved data does not match what is displayed on the screen.

Steps to reproduce:

  • Write two paragraphs.
  • At the end of the second paragraph, press Enter.
  • Now, undo the actions until the entire second paragraph is removed.
  • Right at the moment when the first letter of the second paragraph is removed, the app triggers a refresh, but, it does not remove the letter. If you undo it again at this point, it finally erases the second paragraph and the last letter of the first one.

Environment

  • Editor.js version: 2.27.2
  • Plugins you use with their versions:
  • Device, Browser, OS: MacOS, Chrome

Current behavior

Include code samples, errors, and stack traces if appropriate.
If reporting a bug, please include the reproducing steps.

Expected behavior

When performing an undo while positioned at the first letter of the second paragraph, it is expected that this letter should be deleted.

Additional context

Add any other context about the problem here.

@MarioRodriguezS
Copy link
Contributor

@juanmagiraldor Thanks a lot for reporting the issue, it was solved and it was available in v2.0.24. Please let us know if this solution proves helpful to you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 Bug Something isn't working ⛑ Needs triage Someone is checking
Projects
None yet
Development

No branches or pull requests

3 participants