-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Post editor: fix wrapper block name for template mode #31985
Conversation
8aa26f5
to
71bec7f
Compare
71bec7f
to
48c58c2
Compare
48c58c2
to
baefc18
Compare
Size Change: +16 B (0%) Total Size: 1.62 MB
ℹ️ View Unchanged
|
baefc18
to
d2310c3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, this fixes the issue. 👍 I also tested editing templates, template parts, and reusable blocks, to ensure that those aren't affected (i.e., loops are prevented, but nothing else).
It's clear that injecting context in VisualEditor is not ideal: it's hard to debug and it couples the loop-preventing hook with a particular component. @gziolo and I discussed some possibilities for the near future, including removing any magic from VisualEditor and, at worst, move it into useNoRecursiveRenders.
Thank you for this fix. This functionality needs more work to account for other blocks like the Template Part one. Let's keep it on the radar during 5.8 cycle. |
closes #31869
Props to @ntsekouras for finding the problematic line. :)