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

Holding backspace across paragraph blocks does not delete text #2829

Closed
aduth opened this issue Sep 28, 2017 · 3 comments
Closed

Holding backspace across paragraph blocks does not delete text #2829

aduth opened this issue Sep 28, 2017 · 3 comments
Assignees
@BoardJames
Labels
[Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended

Comments

@aduth
Copy link
Member

aduth commented Sep 28, 2017

Steps to reproduce:

  1. Navigate to Gutenberg > New Post
  2. Insert many paragraph blocks, each with some text
  3. From the last paragraph block, press and hold backspace

Expected result:

All text is deleted.

Actual result:

Text from the removed paragraph appears to be appended to the previous block.

backspace
@aduth aduth added [Priority] High Used to indicate top priority items that need quick attention [Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Type] Bug An existing feature does not function as intended labels Sep 28, 2017
@ellatrix
Copy link
Member

ellatrix commented Oct 2, 2017

Maybe related: At the end of a paragraph, press ENTER twice fast. Two paragraphs are created, but the caret ends up in the top one.

@ellatrix
Copy link
Member

ellatrix commented Oct 2, 2017

Reproducing the GIF, you can even feel that there is a delay and that it's not done in the same call stack.

@BoardJames BoardJames mentioned this issue Nov 13, 2017
Merged
3 tasks
@BoardJames
Copy link

I have a PR for this issue in #3450 . It was caused by the React state not being in sync with TinyMCE's state which in turn was caused by the isDirty function returning an out of date response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BoardJames BoardJames

Labels
[Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@aduth @ellatrix @anna-harrison @BoardJames