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

Dashboard 'Edit styles' link doesn't work #38172

Closed
talldan opened this issue Jan 24, 2022 · 14 comments
Closed

Dashboard 'Edit styles' link doesn't work #38172

talldan opened this issue Jan 24, 2022 · 14 comments
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") Needs Design Feedback Needs general design feedback. [Package] Edit Site /packages/edit-site [Type] Bug An existing feature does not function as intended

Comments

@talldan
Copy link
Contributor

talldan commented Jan 24, 2022

Description

A new user switching to 5.9 will see an 'Edit styles' link on the dashboard encouraging users to check out the new styles feature.

Unfortunately it doesn't work as expected on small / medium sized viewports (phone / tablet). The intention is to launch the site editor with the Styles panel open and welcome guide visible. In testing, the sidebar is only open for a flicker of a moment, before being closed automatically by some responsive logic. The welcome guide also opens but is closed in a flicker, and this means it won't be shown again.

edit: since more recent changes in the Gutenberg plugin, the link doesn't work at all on any screen size. I think these changes will also land in 6.0.

Step-by-step reproduction instructions

1. Clear local storage (specifically the WP_DATA_USER entries)
2. Resize your browser to be 782 pixels or narrower
3. From the Dashboard, click the 'Edit styles' link in the 5.9 banner

Expect: The site editor opens with the styles panel open and a welcome guide visible
Actual: Both the styles panel and welcome guide aren't visible

  1. From the dashboard click 'Edit styles'
  2. Observe that the style sidebar isn't open when the site editor loads.

Screenshots, screen recording, code snippet

Screen Shot 2022-05-10 at 2 19 23 pm

Environment info

Observed while testing 5.9 RC4 backports - WordPress/wordpress-develop#2220
Mac OS / Brave
Gutenberg deactivated

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

@talldan talldan added [Type] Bug An existing feature does not function as intended [Package] Edit Site /packages/edit-site [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") labels Jan 24, 2022
@Andrew-Starr
Copy link

I have the same issue regardless of the viewport size, even on a large screen with browser full-width.

As above the welcome guide flickers visible for a very brief moment and then disappears, and the styles panel is closed.

The URL does have ?styles=open appended, so could there be an issue with this?

@talldan
Copy link
Contributor Author

talldan commented Jan 24, 2022

The URL does have ?styles=open appended, so could there be an issue with this?

@Andrew-Starr That's the thing that makes the styles panel open, but it does seem like something then causes it to close.

What width is your browser window when you experience it? (also, which browser / OS?)

@Andrew-Starr
Copy link

OS: windows 11
Browser: firefox and chrome
browser window is same issue regardless, whether it's 480, 1920 or 3840px

@Andrew-Starr
Copy link

This is the logged WP_DATA_USER local storage data

{"core/edit-site":{"preferences":{"features":{"welcomeGuide":false,"welcomeGuideStyles":true}}}}

@Andrew-Starr
Copy link

Apologies, tested again in a fresh install and it is working now on larger screens.

I can see the issue only on smaller browser widths where there appears to be some strange issue only on those narrower viewports.

Please disregard what I said previously.

@talldan
Copy link
Contributor Author

talldan commented Jan 25, 2022

Thanks for re-testing!

@Andrew-Starr
Copy link

So testing again on a fresh install with core 5.9 and no Gutenberg or other plugins, and all local storage emptied.

Without ?styles=open the site editor loads with a very brief flicker of the welcome guide which then disappears.
Local storage is then set as:
{"core/interface":{"enableItems":{"singleEnableItems":{"complementaryArea":{"core/edit-site":"edit-site/global-styles"}},"multipleEnableItems":{}},"preferences":{"features":{}}},"core/edit-site":{"preferences":{"features":{"welcomeGuide":false,"welcomeGuideStyles":false}}},"core/block-editor":{"preferences":{"insertUsage":{"core/group":{"time":1643190199207,"count":1,"insert":{"name":"core/group"}},"core/group/group-row":{"time":1643190200154,"count":1,"insert":{"name":"core/group/group-row"}}}}}}

With ?styles=open the editor loads with the the styles panel open correctly, but the "welcome to editor styles" welcome guide appears very briefly and then disappears.
Local storage is then set as:
{"core/edit-site":{"preferences":{"features":{"welcomeGuide":false,"welcomeGuideStyles":true}}},"core/block-editor":{"preferences":{"insertUsage":{"core/group":{"time":1643190309589,"count":1,"insert":{"name":"core/group"}},"core/group/group-row":{"time":1643190310730,"count":1,"insert":{"name":"core/group/group-row"}}}}}}

Same behaviour regardless of the browser viewport size. Small screen or large are same issue.

@talldan
Copy link
Contributor Author

talldan commented Apr 19, 2022

Adding this to the 6.0 project board as it was something that was never solved on the 5.9 board.

@ndiego ndiego moved this from Triage to Todo in WordPress 6.0 Editor Tasks Apr 26, 2022
@talldan talldan changed the title Dashboard 'Edit styles' link doesn't work on small / medium viewports Dashboard 'Edit styles' link doesn't work May 10, 2022
@talldan
Copy link
Contributor Author

talldan commented May 10, 2022

It seems like ?styles=open is now stripped from the URL when opening the site editor, so it definitely won't display when following the dashboard link.

@talldan talldan added the Needs Design Feedback Needs general design feedback. label May 10, 2022
@talldan
Copy link
Contributor Author

talldan commented May 10, 2022

Given the dashboard will probably be redesigned for 6.0, I wonder if it'd be better to remove the Edit styles link. It's quite late in the release cycle to fix all the issues (I think there are two or three simultaneous bugs happening that prevent this from working).

@ndiego
Copy link
Member

ndiego commented May 10, 2022

@talldan this is now on the 6.0.1 Project Board. I think we should try and tackle this once and for all in 6.0.1, it's just getting a bit too late for 6.0. Are you ok with that?

@talldan
Copy link
Contributor Author

talldan commented May 11, 2022

@ndiego Yep, agreed 👍

@Mamaduka
Copy link
Member

@talldan
Copy link
Contributor Author

talldan commented Dec 18, 2023

This does seem to almost work again now, though it opens the browse mode styles panel that only has variations.

I think the text on the dashboard isn't quite right as it says "Switch up your site’s look & feel with Styles. Tweak your site, or give it a whole new look! Get creative — how about a new color palette or font?", there isn't that much functionality in this now linked panel, though I believe there are plans to add more, so I think I'll close this.

@talldan talldan closed this as completed Dec 18, 2023
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") Needs Design Feedback Needs general design feedback. [Package] Edit Site /packages/edit-site [Type] Bug An existing feature does not function as intended
Projects
No open projects
Development

No branches or pull requests

4 participants