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

Pattern Assembler: Site View > Styles turns on Site Editor distraction-free mode #79290

Closed
liviopv opened this issue Jul 12, 2023 · 5 comments
Closed
Labels
[Closed] Resolved Issue is resolved. Use for issues that can be closed but did not have an explicit fix with a PR. [Feature] Block Patterns Pattern content itself, and the functionality that lets you create patterns. [Platform] Simple [Pri] Low Address when resources are available. Triaged To be used when issues have been triaged. [Type] Bug When a feature is broken and / or not performing as intended

Comments

@liviopv
Copy link

liviopv commented Jul 12, 2023

Quick summary

After the Assemble Flow is complete and the user is taken to the Site Editor, clicking on the Styles tab in the Site View sidebar will trigger the distraction-free mode in the Site Editor, preventing the user to exit the Styles section without turning off distraction-free mode first.

Video:

CleanShot.2023-07-12.at.10.25.13.mp4

cc @retnonindya @Automattic/lego

Thanks for heads up, @tanya

Steps to reproduce

  1. Use the Pattern Assembler to create a new design. I tested the Assembler in site creation and theme showcase, are there any other entry-points?
  2. Go to the Site Editor
  3. Click on Styles
  4. Distraction-free will be turned on without user input

What you expected to happen

Distraction-free mode shouldn't be turned on without intentional input.

What actually happened

Distraction-free mode is turned on without intentional input.

Impact

Some (< 50%)

Available workarounds?

Yes, easy to implement

Platform (Simple and/or Atomic)

Simple

Logs or notes

Gutenberg 16.1.2 and Gutenberg 16.2 RC3

I haven't tested on Atomic

@liviopv liviopv added [Type] Bug When a feature is broken and / or not performing as intended [Feature] Block Patterns Pattern content itself, and the functionality that lets you create patterns. User Report Needs triage Ticket needs to be triaged labels Jul 12, 2023
@github-actions github-actions bot added [Platform] Simple [Pri] Low Address when resources are available. labels Jul 12, 2023
@tanjoymor
Copy link

Just to clarify, it's not just when using the Assembler flow. It does happen with that flow, because that flow uses Blank Canvas by default which doesn't have style variations. But it happens in the main flow when you pick any theme that doesn't have style variations. It's not always consistent though. My first test with Otis was flipping the Distraction Free mode off and on. When I recorded the process with Skatepark it was off, then came on, then tried to turn itself off but didn't.

RGVckf.mp4

@tanjoymor
Copy link

Related comment here: WordPress/gutenberg#51173 (comment)

@cuemarie
Copy link

📌 SCRUBBING : RESULT - Replicated / Could Not Replicate / Uncertain

  • Tested on Simple – Could Not Replicate

📌 FINDINGS/SCREENSHOTS/VIDEO

  • So far, I haven't been able to replicate this. I've created 2 new sites and tried following the steps above using the Pattern Assembler as well as choosing a theme with no style options, including Skatepark and Rainfall.
  • In all tests, at no point does my browser default to distraction-free editing. 🤔

📌 ACTIONS

  • Requested author feedback

📌 Message to Author

  • @tanjoymor @liviopv What browsers are y'all working in? Any other details that you've found help replicate consistently?

@liviopv
Copy link
Author

liviopv commented Jul 20, 2023

@cuemarie I cannot repro anymore. It could be that 16.2.1 or a different Calypso update fixed it 🙌

@cuemarie
Copy link

Great to hear! In that case, I'll close this out!

📌 ACTIONS

  • Closed as no longer replicable / resolved

@cuemarie cuemarie added Triaged To be used when issues have been triaged. [Closed] Resolved Issue is resolved. Use for issues that can be closed but did not have an explicit fix with a PR. and removed [Status] Needs Author Reply Needs triage Ticket needs to be triaged labels Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Closed] Resolved Issue is resolved. Use for issues that can be closed but did not have an explicit fix with a PR. [Feature] Block Patterns Pattern content itself, and the functionality that lets you create patterns. [Platform] Simple [Pri] Low Address when resources are available. Triaged To be used when issues have been triaged. [Type] Bug When a feature is broken and / or not performing as intended
Projects
None yet
Development

No branches or pull requests

3 participants