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

Styled text not merged using backspace #1873

Closed
cameronvoell opened this issue Feb 7, 2020 · 3 comments · Fixed by #2271
Closed

Styled text not merged using backspace #1873

cameronvoell opened this issue Feb 7, 2020 · 3 comments · Fixed by #2271

Comments

@cameronvoell
Copy link
Contributor

Describe the bug
This issue is a more specific case of #1865 that only occurs if you style text at the front of a paragraph block, and then try and delete some of the styled text, continuing to hit backspace, until it should merge.

To Reproduce

  1. Add a pararaph block to an article and add some text to it
  2. Click in the middle of the text and hit enter to move the right half of the text to a new block
  3. Hit backspace and watch the new block merge back correctly with the previous block
  4. Hit enter from the middle of the text again to create a new block
  5. This time select a continuous group of character from the front of the block, and style them as italic or bold.
  6. Delete one or more characters from the beginning of the new text block, leaving some of the styled text to the right of your cursor.
  7. Continue to hit the backspace key => note that the block is no longer merging with the previous block like it was in step 3

Expected behavior
Backspace at the beginning of a paragraph block should always merge it with a previous paragraph block.

Screenshots
styled-text-no-merge-on-delete

Smartphone (please complete the following information):

  • Pixel 3 Emulator, Android 9
@mchowning
Copy link
Contributor

mchowning commented Apr 2, 2020

I think I ran into another variation on this bug. I found that after splitting blocks if you added an entire word (by swiping), then deleted that word and tried to merge the blocks, they would not merge. If you just tap letters after splitting the blocks though, they will merge.

backspace_merge mp4

Same thing occurs if you just select the first word, delete it, and then try to merge the blocks by tapping delete again:

backspace_split_merge mp4

UPDATE: Tracking this with #2375

@etoledom
Copy link
Contributor

A couple more variations of this bug described here:
wordpress-mobile/WordPress-Android#11572 (comment)

@SergioEstevao
Copy link
Contributor

Confirmed that the bug is also present in iOS too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants