[wp-trac] [WordPress Trac] #42001: Active Widget Not Fully Visible

[WordPress Trac]

#42001: Active Widget Not Fully Visible
-------------------------------------------------+-------------------------
 Reporter:  fervillz                             |       Owner:
                                                 |  ashokrd2013
     Type:  defect (bug)                         |      Status:  assigned
 Priority:  normal                               |   Milestone:  5.3
Component:  Widgets                              |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  good-first-bug has-patch has-        |     Focuses:  javascript
  screenshots needs-refresh                      |
-------------------------------------------------+-------------------------
Changes (by pento):

 * keywords:  good-first-bug has-patch has-screenshots needs-design-feedback
     => good-first-bug has-patch has-screenshots needs-refresh
 * milestone:  5.2 => 5.3


Comment:

 Thanks for the review, @melchoyce!

 I've been doing a little archeology through the codebase, and I have a
 couple of open questions:

 - Why do we still support wide widgets? @shaunandrews, you were the last
 person to [https://github.com/xwp/wp-widget-
 customizer/issues/18#issuecomment-26819319 make a definitive comment] on
 this that I could find.
 - Why are HTML, Text, and RSS widgets wide?

 Regarding the patch:

 - We don't actually need to change the height, since it
 [https://developer.wordpress.org/reference/functions/wp_register_widget_control/
 apparently isn't used].
 - The RSS widget is also wide: if we're updating wide widgets, we should
 do that one, too.

 I'm going to punt this issue to 5.3, I'd appreciate if folks could do some
 more investigation into this.

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