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

Removing empty List block with delete key fails, leads to iOS crash #44990

Closed
dcalhoun opened this issue Oct 14, 2022 · 2 comments · Fixed by #45048
Closed

Removing empty List block with delete key fails, leads to iOS crash #44990

dcalhoun opened this issue Oct 14, 2022 · 2 comments · Fixed by #45048
Assignees
Labels
[Block] List Affects the List Block Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) [Priority] High Used to indicate top priority items that need quick attention [Type] Regression Related to a regression in the latest release

Comments

@dcalhoun
Copy link
Member

Description

When attempting to remove an empty List block by pressing the keyboard delete key, the block is not removed on Android and the app crashes on iOS.

Step-by-step reproduction instructions

  1. Add a List block to the content.
  2. Pressing the Delete key.

Expected behaviour

The List block is removed and replaced with a Paragraph block.

Actual behaviour

Android: The List block is not removed nor replaced.
iOS: The app crashes.

Screenshots or screen recording (optional)

Android Screencast
Screen_Recording_20221014-145708_WordPress.Pre-Alpha.mp4
iOS Screencast
RPReplay_Final1665777602.MP4

WordPress information

  • WordPress version: 6.1
  • Gutenberg version: 14.3.0
  • Are all plugins except Gutenberg deactivated? Yes
  • Are you using a default theme (e.g. Twenty Twenty-One)? Geologist

Device information

  • Device: Samsung Galaxy S20, iPhone SE
  • Operating system: Android 12, iOS 16.1
  • WordPress app version: 1.84.0 or soon-to-be 20.10 beta
@dcalhoun dcalhoun added [Priority] High Used to indicate top priority items that need quick attention [Type] Regression Related to a regression in the latest release Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) [Block] List Affects the List Block labels Oct 14, 2022
@fluiddot
Copy link
Contributor

I tested on version 20.9 and the issue can't be reproduced, so definitely this is a regression on version 21.0.

@dcalhoun dcalhoun self-assigned this Oct 17, 2022
@dcalhoun
Copy link
Member Author

This regression appears to have originated from #43181. I am researching further.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] List Affects the List Block Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) [Priority] High Used to indicate top priority items that need quick attention [Type] Regression Related to a regression in the latest release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants