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 widget editor on .com enters constant refresh loop #53448

Closed
simison opened this issue Jun 7, 2021 · 3 comments
Closed

Block widget editor on .com enters constant refresh loop #53448

simison opened this issue Jun 7, 2021 · 3 comments
Assignees
Labels
[Closed] Fixed Issues explicitly fixed with PRs. Do not use for no longer reproducible issues. [Goal] Block widget editor Block based Widgets Editor coming in WP 5.8 [Type] Bug

Comments

@simison
Copy link
Member

simison commented Jun 7, 2021

This can be harder to replicate, but we're seeing the experimental block widget editor on .com go into a refresh-loop:

screen-capture-on-2021-06-01-at-09-39-15_mp4_hd.mp4
screen-recording-2021-05-31-at-16.09.26_mov_hd.mp4

via p58i-axR-p2/#comment-50850

The moment I go to Appearance > Widgets is when I experience the constant refresh of the Widgets Editor screen.

I can click within a widget to attempt to make changes but the screen will still refresh. The Widgets Editor sticker is enabled for the site in the BlogRC. This seems to occur whether or not the “Show wp-admin pages if available” toggle is on.

The feature is going to be released in WP 5.8 July by which time we should have this fixed, although we might want to release the feature earlier on .com.

@simison simison added [Type] Bug [Goal] Block widget editor Block based Widgets Editor coming in WP 5.8 labels Jun 7, 2021
@simison
Copy link
Member Author

simison commented Jun 7, 2021

Block editor has a similar issue (#53448) but Jacopo mentioned (p58i-aCg-p2#comment-50871) it doesn't seem exactly the same:

Looking at the various videos, I’m kind of inclined to think the Site Editor one is a routing loop; at least the Widget Editor loads! 🤔

@taggon taggon self-assigned this Jun 8, 2021
@taggon
Copy link
Contributor

taggon commented Jun 8, 2021

After hours of investigation, I found the root cause is one of the legacy widget hooks not called. When it comes to iframe request, the hook is supposed to define IFRAME_REQUEST to prevent wpcom-frame-handling mu-plugin from reloading the top frame over and over again. But it is not called due to the order of hooks.

While working on the fix, I'm noticed the fix already has been merged to the Gutenberg project. What we need is only checkpicking. :)

@simison
Copy link
Member Author

simison commented Jun 8, 2021

Oh, fantastic! Good sleuthing.

The fix is included in Gutenberg 10.8 which seems fine to wait (should be in a week?). If cherry-picking that all-PHP change is easy enough, that's also worth it. FYI @fullofcaffeine @WunderBart

FYI @noisysocks @draganescu that the above PR fixes this refresh issue on .com.

@simison simison added the [Closed] Fixed Issues explicitly fixed with PRs. Do not use for no longer reproducible issues. label Jun 8, 2021
@simison simison closed this as completed Sep 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Closed] Fixed Issues explicitly fixed with PRs. Do not use for no longer reproducible issues. [Goal] Block widget editor Block based Widgets Editor coming in WP 5.8 [Type] Bug
Projects
None yet
Development

No branches or pull requests

2 participants