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

Add Block prompt regression #31562

Closed
annezazu opened this issue May 6, 2021 · 5 comments · Fixed by #31882
Closed

Add Block prompt regression #31562

annezazu opened this issue May 6, 2021 · 5 comments · Fixed by #31882
Assignees
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Feature] Template Editing Mode Related to the template editor available in the Block Editor General Interface Parts of the UI which don't fall neatly under other labels. Needs Design Feedback Needs general design feedback. [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended [Type] Regression Related to a regression in the latest release

Comments

@annezazu
Copy link
Contributor

annezazu commented May 6, 2021

Description

As reported by others in core-editor slack , there seems to be a regression in the add block prompt within the site editor. Rather than just showing a + button, there is now text saying "Add Block".

Step-by-step reproduction instructions

  1. Set up a site with FSE capabilities
  2. Use either the site editor or template editing mode and notice that the add block prompt has changed.

Expected behaviour

Expect to just see the + button and no "Add Block" text.

Actual behaviour

The add block appender now say "add block" and looks quite awkward :D

Screenshots or screen recording (optional)

Here are two examples in the Site Editor where adding a new template part causes it and trying to add a new navigation block item:

add.block.mov

Here's an example from the Template Editing Mode:

add.block.template.editing.mode.mov

WordPress information

  • WordPress version: 5.7.1
  • Gutenberg version: 10.6RC
  • Are all plugins except Gutenberg deactivated? Yes
  • Are you using a default theme (e.g. Twenty Twenty-One)? TT1 Blocks

Device information

  • Device: Desktop
  • Operating system: MacOS
  • Browser: Chrome
@annezazu annezazu added General Interface Parts of the UI which don't fall neatly under other labels. Needs Design Feedback Needs general design feedback. [a11y] Color Contrast labels May 6, 2021
@annezazu annezazu changed the title Add Block prompt blends in with dark background Add Block prompt regression May 6, 2021
@annezazu annezazu added [Type] Bug An existing feature does not function as intended [Type] Regression Related to a regression in the latest release [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Feature] Template Editing Mode Related to the template editor available in the Block Editor and removed [a11y] Color Contrast labels May 6, 2021
@jasmussen
Copy link
Contributor

Definitely a bug, inspecting this I see that there's a "visuallyhidden" component, which is used when the text is meant as added context for screenreaders, but not meant to be seen:

Screenshot 2021-05-07 at 10 29 45

Since the bug is present only in the site editor, I wonder if the regression is related to the split block styles loading system that landed recently, but only for block themes. CC: @aristath

@aristath
Copy link
Member

aristath commented May 7, 2021

v10.5 works as expected, the regression is in v10.6 and the block-styles tweaks far predate these 2 versions 😉
Using git bisect I tracked down the issue to this PR: #31244

@jasmussen
Copy link
Contributor

Thanks so much for doing that. CC @sarayourfriend in case you might have thoughts.

@sarayourfriend
Copy link
Contributor

This is a problem with iframe support in Emotion. It seems to work in some cases (if you try a block like the Jetpack Google Calendar block you can see that the external link is rendered with the correct CSS-in-JS styles) but for some reason it's not working for the columns block appender. I'm not familiar with the architecture of FSE so I don't know why one is inside the StyleProvider and the other isn't. Maybe @youknowriad will know?

@youknowriad
Copy link
Contributor

I think all of these are supposed to be already inside a StyleProvider 🤔 It's not a Slot AFAIK

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") [Feature] Template Editing Mode Related to the template editor available in the Block Editor General Interface Parts of the UI which don't fall neatly under other labels. Needs Design Feedback Needs general design feedback. [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended [Type] Regression Related to a regression in the latest release
Projects
None yet
5 participants