[wp-trac] [WordPress Trac] #29790: Widgets don't know the widget area context there in

WordPress Trac noreply at wordpress.org
Thu Oct 2 13:03:10 UTC 2014


#29790: Widgets don't know the widget area context there in
------------------------------+--------------------------------------------
 Reporter:  ruud@…            |       Owner:
     Type:  enhancement       |      Status:  new
 Priority:  normal            |   Milestone:  Awaiting Review
Component:  Widgets           |     Version:  trunk
 Severity:  normal            |  Resolution:
 Keywords:  has-patch dev-    |     Focuses:  accessibility, administration
  feedback                    |
------------------------------+--------------------------------------------

Comment (by ruud@…):

 Hi Weston. Your suggestion is valid indeed.
 I know we can leverage the widget callbacks to obtain the widget area
 information, that's how we currently implemented it for our use case.

 However, I think this is perhaps not common knowledge and adding some
 additional information to the $instance seems rather harmless and opens up
 possibilities in an easy way for widget developers.

 So to get more to the point on this: is it your opinion that because a
 developer can achieve this via alternative routes it should not go into
 core? Or is the way (by adding it to the $instance variable) not right?
 Don't get this the wrong way, I just want your opinion on this and I
 personally don't know what should or shouldn't go into core, and on what
 grounds we can make a decision on this as well.
 I mean: should we focus on light weight code, or on ease of use, or on ??

 Thanks.

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


More information about the wp-trac mailing list