[wp-trac] [WordPress Trac] #48734: Twenty Twenty: [em] tag with blank string inside ruins theme layout

WordPress Trac noreply at wordpress.org
Mon Jun 24 13:28:36 UTC 2024


#48734: Twenty Twenty: [em] tag with blank string inside ruins theme layout
-------------------------------------------------+-------------------------
 Reporter:  derlynad                             |       Owner:  audrasjb
     Type:  defect (bug)                         |      Status:  accepted
 Priority:  normal                               |   Milestone:  Future
                                                 |  Release
Component:  Bundled Theme                        |     Version:  5.3
 Severity:  normal                               |  Resolution:
 Keywords:  has-screenshots has-patch needs-     |     Focuses:
  testing 2nd-opinion                            |
-------------------------------------------------+-------------------------
Changes (by sabernhardt):

 * keywords:  has-screenshots has-patch needs-testing => has-screenshots
     has-patch needs-testing 2nd-opinion


Comment:

 Thanks for updating the patch. I'm not so sure about setting `display:
 block` now.

 Having a `strong` or `em` element without a parent `div` or paragraph is
 already an edge case, and the situation described on this ticket is
 actually caused by invalid HTML.

 The case of someone actually wanting parent-less inline elements to
 continue displaying inline is likely even less common, but it technically
 is possible. Adding `text <em>without</em> paragraph` in a Custom HTML
 block did not honor the margin, and `display: block` made it more broken
 than it already was.

 Also, the numbers I mentioned earlier are the changesets that made the
 first patch unusable, not ticket numbers.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/48734#comment:16>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list