[wp-trac] [WordPress Trac] #53801: Block-based Widgets Screen does action wp_footer after each Widget

WordPress Trac noreply at wordpress.org
Sat Sep 4 01:50:29 UTC 2021


#53801: Block-based Widgets Screen does action wp_footer after each Widget
-------------------------------------+---------------------
 Reporter:  MadtownLems              |       Owner:  (none)
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  5.8.2
Component:  Widgets                  |     Version:  5.8
 Severity:  major                    |  Resolution:
 Keywords:  has-patch needs-testing  |     Focuses:
-------------------------------------+---------------------

Comment (by eatingrules):

 Replying to [comment:28 costdev]:
 > I've added an updated patch that contains the additional rules.

 I just tested the patch on a site that was having these issues and it
 worked well. It hid three different script-based items that were getting
 in the way.🎉

 Nevertheless, the previews still looked pretty bad since the block width
 is much wider than the actual sidebar width. (I realize this is a separate
 issue...I searched trac briefly but didn't find an open ticket on that
 yet?)

 My preference would still be to avoid legacy widget previews altogether,
 for reasons I stated earlier in this thread... especially the "mode
 shifting" concerns, which I still find jarring.

 > Now that legacy widget previews have already been released, I think
 removing them is no longer an option.

 I don't see a problem with going back on the legacy widget previews, since
 it won't break anything to stop showing previews.

 Alternatively, an option to disable the legacy widget preview would be a
 good "failsafe" so that if the proposed patch doesn't catch all
 situations, at least then the user could toggle the previews off.

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


More information about the wp-trac mailing list