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

Inconsistent Pattern Insertion between Page Editor and Site Editor #31251

Closed
ianstewart opened this issue Apr 27, 2021 · 2 comments · Fixed by #31448
Closed

Inconsistent Pattern Insertion between Page Editor and Site Editor #31251

ianstewart opened this issue Apr 27, 2021 · 2 comments · Fixed by #31448
Assignees
Labels
Needs Testing Needs further testing to be confirmed.

Comments

@ianstewart
Copy link
Contributor

Description

The site editor has a pattern insertion experience inconsistent with the page editor. I think, likely, due to what happens when one opens the top inserter in the site editor.

Step-by-step reproduction instructions

Compare the following pattern insertion experiences.

Page Editor

insert-pattern-into-page-small

Site Editor

insert-pattern-into-template-small

Expected behaviour

One would be able to see the pattern they just inserted into the site editor.

Actual behaviour

When inserting a pattern into the bottom of a template I was not able to see the pattern I just inserted without scrolling down.

WordPress information

  • WordPress version: 5.7.1
  • Gutenberg version: 10.4.1
  • Are all plugins except Gutenberg deactivated? No. (Jetpack active.)
  • Are you using a default theme (e.g. Twenty Twenty-One)? No. (Hansen.)

Device information

  • Device: Desktop
  • Operating system: MacOS 11.3
  • Browser: Chrome 91.0.4472.19
@ntsekouras ntsekouras added the Needs Testing Needs further testing to be confirmed. label Apr 28, 2021
@ntsekouras
Copy link
Contributor

Hey 👋 I couldn't reproduce on latest trunk...

patterns

@david-szabo97
Copy link
Member

I can reproduce the issue.

Screen.Recording.2021-05-03.at.10.55.11.mov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Testing Needs further testing to be confirmed.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants