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 Library]: Remove unnecessary usages of RawHTML
#38527
[Block Library]: Remove unnecessary usages of RawHTML
#38527
Conversation
Size Change: +24 B (0%) Total Size: 1.14 MB
ℹ️ View Unchanged
|
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.
All cases but one look good.
d16e2bb
to
a2c43b9
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.
I think this looks good now. I haven't tested the Latests Posts block yet. Everything else looks straightforward.
Related: #38451
It seems in some cases we don't need to use
RawHTML
component as it creates an extradiv
. This PR just does that and usesdangerouslySetInnerHTML
directly - which is whatRawHTML
uses anyway.Testing instructions
Query Loop
block add thePost Title
block (this PR affects only the readonly view) and observe in the dev tool the html markup that an extradiv
is not rendered.Latest Posts
block and observe the same as above in dev tools for the post title andfull
content(have to enable it through inspector controls).This PR doesn't change anything in the front end.