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

Closed
stokesman opened this issue Jun 12, 2024 · 10 comments · Fixed by #64789
Closed

Site editor: Welcome Guide apparition #62525

stokesman opened this issue Jun 12, 2024 · 10 comments · Fixed by #64789
Assignees
Labels
[Package] Edit Site /packages/edit-site [Status] In Progress Tracking issues with work in progress [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 colorful-tones moved this from ❓ Triage to 📥 Todo in WordPress 6.6 Editor Tasks 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 🤔

@ellatrix ellatrix moved this from 📥 Todo to ❓ Triage in WordPress 6.6 Editor Tasks Jul 9, 2024
@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.

Copy link

Hi,
This issue has gone 30 days without any activity. This means it is time for a check-in to make sure it is still relevant. If you are still experiencing this issue with the latest versions, you can help the project by responding to confirm the problem and by providing any updated reproduction steps.
Thanks for helping out.

@github-actions github-actions bot added the [Status] Stale Gives the original author opportunity to update before closing. Can be reopened as needed. label Aug 15, 2024
@stokesman
Copy link
Contributor Author

Just saying this still happens.

@github-actions github-actions bot removed the [Status] Stale Gives the original author opportunity to update before closing. Can be reopened as needed. label Aug 23, 2024
@ramonjd
Copy link
Member

ramonjd commented Aug 23, 2024

I can reproduce - I think it's important that there are no user preferences stored in the browser (the preference key is welcomeGuideTemplate), hence the private window test scenario mentioned above.

I had to screenshot a frame of my screencast as the optimized video reduced the frame rate 😄

Screenshot 2024-08-23 at 1 07 29 PM

As far as I can tell, the template welcome guide will show when the store subscription callback thinks that the current page is a page type and the current post type is a template, among other conditions:

return (
isTemplateActive &&
! isEditorActive &&
isPage() &&
getCurrentPostType() === 'wp_template'
);

Here's what I'm logging from the selector when I follow the test instructions:

{isTemplateActive: true, isEditorActive: false, isPage: false, currentPostType: 'wp_template'}

{isTemplateActive: true, isEditorActive: false, isPage: true, currentPostType: 'wp_template'} // Flash here - it satisfies the conditions

{isTemplateActive: true, isEditorActive: false, isPage: true, currentPostType: 'page'}

@stokesman
Copy link
Contributor Author

Ramon, thanks for testing and inspecting this. I looked into this further and it appears that the flash of the template welcome guide is only half the issue. The other half is that it never presents as originally implemented in #52014. From what I see there, it should appear when one has chosen to edit a template starting from editing a page. I can’t get that to happen on trunk or testing WP 6.6 on Playground. The latter surprised me because #52014 was apparently backported to 6.6 and should have been working.

I’ve spun up #64789 aimed to resolve this.

@stokesman stokesman removed the Needs Testing Needs further testing to be confirmed. label Aug 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package] Edit Site /packages/edit-site [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended
Projects
No open projects
Status: Triage
Development

Successfully merging a pull request may close this issue.

5 participants