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

Block preview when adding a block is not working in WordPress 5.6.1 #28823

Closed
blankart opened this issue Feb 8, 2021 · 16 comments
Closed

Block preview when adding a block is not working in WordPress 5.6.1 #28823

blankart opened this issue Feb 8, 2021 · 16 comments
Labels
[Feature] Inserter The main way to insert blocks using the + button in the editing interface [Type] Bug An existing feature does not function as intended

Comments

@blankart
Copy link

blankart commented Feb 8, 2021

Description

Block preview when adding a block is not working in WordPress 5.6.1

Step-by-step reproduction instructions

Click the "add block" button. Hover one of the block entry buttons.

Expected behaviour

A popover with block preview should be visible

Actual behaviour

The popover is not visible

Screenshots or screen recording (optional)

Screen Shot 2021-02-08 at 10 29 53 AM

WordPress information

  • WordPress version: 5.6.1
  • Gutenberg version: Not installed
  • Are all plugins except Gutenberg deactivated? Yes
  • Are you using a default theme (e.g. Twenty Twenty-One)? Yes

Device information

  • Device: Desktop
  • Operating system: MacOS Big Sur
  • Browser: Chrome
@gziolo gziolo added Needs Testing Needs further testing to be confirmed. [Feature] Inserter The main way to insert blocks using the + button in the editing interface labels Feb 8, 2021
@paaljoachim
Copy link
Contributor

paaljoachim commented Feb 8, 2021

Thank you for the report @blankart

Testing.
Theme: Twenty Twenty One.
WordPress version: 5.6.1
No plugins activated.
Chrome
Mac OSX 10.15.7 (Catalina)

I open an existing page (draft).
No-preview-blocks

Twenty Nineteen and Twenty Twenty theme also contains the same lack of block preview error.
The same as with TT1 theme.
Tested with estar theme. Same error.


Activating the Gutenberg plugin 9.9.1 the Block preview works.

Screen Shot 2021-02-08 at 12 56 41


Testing 5.6 version of WordPress.
No plugins activated.

Screen Shot 2021-02-08 at 13 00 48

Block preview works in 5.6.


My conclusion is that the 5.6.1 update has caused an error so that Block previews are not seen.

@noisysocks Robert.

@noisysocks
Copy link
Member

@paaljoachim: Could you confirm if this happens in 5.7 beta 1?

@paaljoachim
Copy link
Contributor

Good call, Robert!
@noisysocks

Testing.
Theme: Twenty Twenty One.
WordPress version: 5.7 beta 1
Only the WordPress Beta Tester plugin is activated.
Chrome
Mac OSX 10.15.7 (Catalina)

Opening Sample Page. Changed it to a draft.
Block previews work fine in 5.7 beta 1.

Block-preview-5-7-beta1

Tested:
Existing page and post.
New page and post.
Draft page and post.
Block preview works nicely.

@tobiasger
Copy link

I'm having the same issue with Wordpress Version 5.6.1.

@paaljoachim paaljoachim added [Type] Bug An existing feature does not function as intended and removed Needs Testing Needs further testing to be confirmed. labels Feb 10, 2021
@bobbingwide
Copy link
Contributor

Ditto. But shouldn't there be a TRAC issue since this problem occurs when Gutenberg is not activated.

@paaljoachim
Copy link
Contributor

Hi Riad and Greg. I am just informing you about this issue.

@youknowriad @gziolo

@youknowriad
Copy link
Contributor

Do we know if there will be a 5.6.2 before 5.7. It's not clear at this point? and this is already fixed ini 5.7 so.
It would also be good to try to find what fixed this issue.

@paaljoachim
Copy link
Contributor

paaljoachim commented Feb 12, 2021

There are talks about a 5.6.2 release before the release of 5.7.
I asked in core for an official P2 post about the update.

@paaljoachim paaljoachim added the Backport to WP Minor Release Pull request that needs to be backported to a WordPress minor release label Feb 14, 2021
@desrosj
Copy link
Contributor

desrosj commented Feb 16, 2021

I'm working on preparing a 5.6.2 RC, and hoping to get that out at some point today (the 16th). Has anyone been able to locate the change that fixed this issue? I'll take a look, but without being able to find that, we may have to skip this one in 5.6.2.

@ockham
Copy link
Contributor

ockham commented Feb 16, 2021

@desrosj The wp/5.6 branch has a list of commits that should allow narrowing the search down (as to what broke block preview). Given the release dates of WP 5.6.1 (Feb 3) and 5.6.0 (Dec 8), I believe that that commit must be one of the top 8 ones that were apparently cherry-picked on Jan 28.

@ockham
Copy link
Contributor

ockham commented Feb 16, 2021

#26432 seems like a likely candidate; #26583's PR desc says

The fix in #26432 accidentally broke block previews in the Inserter due to a naïve use of overflow: hidden to fix the overscroll with the block library open in the site editor.

While it only refers to the site editor (and not the post editor), the files that it touches seem editor agnostic.

cc/ @mattwiebe for confirmation 😊

Maybe someone (@desrosj or @paaljoachim? 🙏 ) could give this a try. (It's a bit late here, I'm already after hours 😉 )

@desrosj
Copy link
Contributor

desrosj commented Feb 16, 2021

Thanks for the assist, @ockham! #26583 does indeed restore block previews in the 5.6 branch! I've gone ahead and added that to #29042.

@tellthemachines
Copy link
Contributor

I just tested #26583 on the 5.6 branch and it works well, though I see @desrosj beat me to it 😄

@ockham
Copy link
Contributor

ockham commented Feb 17, 2021

Filed #29073 to request an e2e test for block preview visibility.

@desrosj
Copy link
Contributor

desrosj commented Feb 17, 2021

This was merged in #29058 and will be shipped in the upcoming 5.6.2.

@desrosj desrosj closed this as completed Feb 17, 2021
@desrosj desrosj removed the Backport to WP Minor Release Pull request that needs to be backported to a WordPress minor release label Feb 17, 2021
@mattwiebe
Copy link
Contributor

While it only refers to the site editor (and not the post editor), the files that it touches seem editor agnostic

Yes, you are correct, this touched the interface skeleton that is used to construct both post and site editors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Inserter The main way to insert blocks using the + button in the editing interface [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests