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

Inserter: nested overflow on patterns tab #65837

Closed
2 tasks done
stokesman opened this issue Oct 2, 2024 · 2 comments · Fixed by #65387
Closed
2 tasks done

Inserter: nested overflow on patterns tab #65837

stokesman opened this issue Oct 2, 2024 · 2 comments · Fixed by #65387
Assignees
Labels
[Type] Regression Related to a regression in the latest release

Comments

@stokesman
Copy link
Contributor

Description

There’s an extra scrolling area appearing in the inserter on the patterns tab (probably the media tab too if it had enough content to overflow).

It seems to have come about after #64371 (tested by checking out the parent commit b7bf133 and noting the lack of the issue). It may be that it’s no fault of that PR’s changes and some detail of this use case is "doing it wrong".

Step-by-step reproduction instructions

  1. Have a look at the patterns tab with a window short enough that there’s overflow

Screenshots, screen recording, code snippet

inserter-nested-overflow.mp4

Environment info

No response

Please confirm that you have searched existing issues in the repo.

  • Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

  • Yes
@stokesman stokesman added the [Type] Regression Related to a regression in the latest release label Oct 2, 2024
@ciampo
Copy link
Contributor

ciampo commented Oct 2, 2024

Is #65387 fixing this issue for you?

cc @DaniGuardiola

@DaniGuardiola
Copy link
Contributor

I fixed this issue in #65387

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Regression Related to a regression in the latest release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants