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

Certain Buttons Unresponsive in Site Editor on Mobile Browsers and Apps #58781

Closed
SiobhyB opened this issue Feb 7, 2024 · 5 comments · Fixed by #58852
Closed

Certain Buttons Unresponsive in Site Editor on Mobile Browsers and Apps #58781

SiobhyB opened this issue Feb 7, 2024 · 5 comments · Fixed by #58852
Assignees
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Package] Edit Site /packages/edit-site [Priority] High Used to indicate top priority items that need quick attention [Status] In Progress Tracking issues with work in progress [Type] Regression Related to a regression in the latest release

Comments

@SiobhyB
Copy link
Contributor

SiobhyB commented Feb 7, 2024

Description

As reported in Automattic/wp-calypso#87247 by @kelasante, buttons that appear in the top toolbar when editing a template in the site editor are not tappable. Nothing happens when you click on a specific feature like the save or style icon.

Markup 2024-02-06 at 17 39 22

Additionally, when you scroll to Patterns, and click to open a pattern, nothing happens.

Markup 2024-02-06 at 17 42 44

Step-by-step reproduction instructions

  1. On a mobile app or mobile browser on a site with an FSE block theme active, open the Site Editor by clicking the option to update your site's design.
  2. Go to templates
  3. Click on any of the buttons at the top of the page, like save or the block inserter
  4. Similarly, go to Patterns, and click or double-click on a pattern to open it.

Screenshots, screen recording, code snippet

No response

Environment info

This is a new regression that's reproducible with 17.6.3 of the Gutenberg plugin.

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

@SiobhyB SiobhyB added [Type] Bug An existing feature does not function as intended [Priority] High Used to indicate top priority items that need quick attention [Type] Regression Related to a regression in the latest release [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") labels Feb 7, 2024
@SiobhyB
Copy link
Contributor Author

SiobhyB commented Feb 7, 2024

Noting that the buttons become responsive again after changing the z-index for the following selector locally:

.edit-site-layout__header {
height: $header-height;
display: flex;
z-index: z-index(".edit-site-layout__header");
// This is only necessary for the exit animation
.edit-site-layout:not(.is-full-canvas) & {
position: fixed;
width: 100vw;
}
}

There appear to have been a number of changes related to the site editor which may have contributed.

@bph bph added the [Block] Buttons Affects the Buttons Block label Feb 7, 2024
@afercia afercia added [Package] Edit Site /packages/edit-site and removed [Type] Bug An existing feature does not function as intended [Block] Buttons Affects the Buttons Block labels Feb 7, 2024
@afercia
Copy link
Contributor

afercia commented Feb 7, 2024

I noticed this regression as well and mentioned in another PR, see #58740

Note: the layout suffers from an unrelated bug where some transparent elements overlap the top bar button. As such it's not possible to click these buttons. Use the keyboard to move to them.

I don't think this is related to the Block buttons and not even to the Button component. It's the SIte editor top bar layout that needs some adjustments

@t-hamano
Copy link
Contributor

t-hamano commented Feb 7, 2024

Additionally, when you scroll to Patterns, and click to open a pattern, nothing happens.

This issue is also mentioned in this comment: #58044 (comment)

@getdave
Copy link
Contributor

getdave commented Feb 8, 2024

I confirmed this Issue is happening. I believe this selector is responsible:

.edit-site-layout__header {
height: $header-height;
display: flex;
z-index: z-index(".edit-site-layout__header");

Specifically the z-index (which resolved to 2). If I bump that up to 3 I can click on the buttons but the site logo becomes obscured:

Screen.Capture.on.2024-02-08.at.16-44-53.mp4

@jeryj
Copy link
Contributor

jeryj commented Feb 8, 2024

I have a bugfix ready for review: #58852

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Package] Edit Site /packages/edit-site [Priority] High Used to indicate top priority items that need quick attention [Status] In Progress Tracking issues with work in progress [Type] Regression Related to a regression in the latest release
Projects
Status: Done
6 participants