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

Site editor: Welcome Guide apparition #62525

Open
stokesman opened this issue Jun 12, 2024 · 6 comments
Open

Site editor: Welcome Guide apparition #62525

stokesman opened this issue Jun 12, 2024 · 6 comments
Labels
Needs Testing Needs further testing to be confirmed. [Package] Edit Site /packages/edit-site [Type] Bug An existing feature does not function as intended

Comments

@stokesman
Copy link
Contributor

stokesman commented Jun 12, 2024

Description

In trunk and the site editor I'm seeing a brief flash of the template editing Welcome Guide anytime I’ve reloaded the site and go from view mode to edit a page. It doesn’t happen on subsequent switches between the modes.

It works as expected if I create a new user and login. So it probably works right for fresh installs too.

Step-by-step reproduction instructions

  1. Start from the WP admin Dashboard.
  2. Visit the site editor.
  3. Go to "Pages" view
  4. Use the "Edit" button (pencil icon) to edit the page directly (without first having selected the page)
  5. Watch for the flash of the Welcome Guide

Screenshots, screen recording, code snippet

welcome-guide-apparition.mp4

Environment info

  • Trunk
  • Chrome
  • macOS

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 [Type] Bug An existing feature does not function as intended [Package] Edit Site /packages/edit-site labels Jun 12, 2024
@annezazu
Copy link
Contributor

I tried replicating with 6.6-beta2-58426 using WordPress nightly on Playground and couldn't. Going to add it to the 6.6 board to be safe but curious if others can replicate.

@stokesman
Copy link
Contributor Author

Thanks for testing and putting it on the board. I’m not sure it will be reproducible on Playground as I tested and did’t see any Welcome Guide ever. I thought that was supposed to act like a fresh install so that’s puzzling to me. If no Welcome Guides are shown perhaps we can’t expect the apparition either.

I’m still seeing it on trunk though I've realized there’s another requirement to reproduce. When clicking the "edit" button of a page, the preview has to be showing the initial "home" page. I.e. the page you are about to edit shouldn’t be highlighted/selected. This quite obscure so probably not much of a priority.

@colorful-tones colorful-tones added the Needs Testing Needs further testing to be confirmed. label Jun 20, 2024
@colorful-tones
Copy link
Member

I attempted to recreate this bug against 7201eec and with WP 6.6 beta 3 (WP Beta Tester plugin), but I'm not certain I'm following the reproducible steps?

I tested in Chrome 126 with cache disabled and Network throttling for Slow 3G and Fast 3G while clicking through various site editor screens and I was not able to reproduce it.

I’m still seeing it on trunk though I've realized there’s another requirement to reproduce. When clicking the "edit" button of a page, the preview has to be showing the initial "home" page. I.e. the page you are about to edit shouldn’t be highlighted/selected. This quite obscure so probably not much of a priority.

Do you mind providing updated testing steps to recreate the issue please? I'm also wondering if this is impacted by define( 'SCRIPT_DEBUG', true );?

@stokesman
Copy link
Contributor Author

Thanks for testing.

Do you mind providing updated testing steps to recreate the issue please?

I don’t mind but I’m not able to see any missing step. I revised them a bit to hopefully make it more clear. If it helps here’s another repro video (this one from 5738e9b):

welcome-guide-apparition-2.mp4

I'm also wondering if this is impacted by define( 'SCRIPT_DEBUG', true );?

Good query. I checked again and can reproduce either way.

@talldan
Copy link
Contributor

talldan commented Jun 24, 2024

I also tried, but can't reproduce this 🤔

@stokesman
Copy link
Contributor Author

stokesman commented Jul 15, 2024

Back here to report that I was able to reproduce this while testing in Playground. I had to visit in a private window to have any Welcome Guides showing up and had to visit the editor once to dismiss the expected ones. After that, exiting to the dashboard and following the reproduction steps worked. Though, I do think sometimes it’s not visible probably due to painting performance variations in the browser.

Screen.Recording.2024-07-15.at.10.06.49.AM.mp4

Also the reason I tested Playground again was I happened to notice elsewhere @annezazu had reproduced this on Playground #61489 (comment), it can be see in that video 11 seconds into it.

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. [Package] Edit Site /packages/edit-site [Type] Bug An existing feature does not function as intended
4 participants