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

Ungroup option displayed in deeply nested block warning doesn't work properly #55738

Closed
fluiddot opened this issue Oct 31, 2023 · 0 comments · Fixed by #56445
Closed

Ungroup option displayed in deeply nested block warning doesn't work properly #55738

fluiddot opened this issue Oct 31, 2023 · 0 comments · Fixed by #56445
Assignees
Labels
Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended

Comments

@fluiddot
Copy link
Contributor

Description

Related to #54382, depending on the block types deeply nested, the ungroup option doesn't work.

Step-by-step reproduction instructions

  1. Create/open a post in the app.
  2. Copy the following HTML code with a deeply nested structure.
Click here to display the HTML code
<ul>
    <li>1
        <ul>
            <li>2
                <ul>
                    <li>3<ul>
                            <li>4<ul>
                                    <li>5<ul>
                                            <li>6<ul>
                                                    <li>7<ul>
                                                            <li>8<ul>
                                                                    <li>9</li>
                                                                </ul>
                                                            </li>
                                                        </ul>
                                                    </li>
                                                </ul>
                                            </li>
                                        </ul>
                                    </li>
                                </ul>
                            </li>
                        </ul>
                    </li>
                </ul>
            </li>
        </ul>
    </li>
</ul>
  1. Paste the content in a Paragraph block.
  2. Observe that a List block is created and that the "Block cannot be rendered ..." warning is displayed.
  3. Tap on the warning message two times until a bottom sheet is displayed.
  4. Tap on the "Ungroup block" option.
  5. Observe that no action is performed and that the bottom sheet remains open.

Expected behaviour

The ungroup block option should work, or if ungrouping is unavailable, remove the option from the bottom sheet.

Actual behaviour

The ungroup block option is always displayed and doesn't work on specific cases.

Screenshots or screen recording (optional)

WordPress information

  • WordPress version: N/A
  • Gutenberg version: N/A
  • Are all plugins except Gutenberg deactivated? N/A
  • Are you using a default theme (e.g. Twenty Twenty-One)? N/A

Device information

  • Device: iPhone 15 (simulator)
  • Operating system: iOS 17.0
  • WordPress app version: 23.5
@fluiddot fluiddot added [Type] Bug An existing feature does not function as intended Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) labels Oct 31, 2023
@fluiddot fluiddot self-assigned this Nov 2, 2023
@github-actions github-actions bot added the [Status] In Progress Tracking issues with work in progress label Nov 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant