[wp-trac] [WordPress Trac] #32470: Abstracting the Widget Classes
WordPress Trac
noreply at wordpress.org
Mon May 25 13:54:55 UTC 2015
#32470: Abstracting the Widget Classes
--------------------------+------------------------------
Reporter: welcher | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Widgets | Version:
Severity: normal | Resolution:
Keywords: dev-feedback | Focuses:
--------------------------+------------------------------
Comment (by jdgrimes):
Replying to [comment:11 welcher]:
> Replying to [comment:10 jdgrimes]:
> > I think that probably in 90% of cases that's the only one that would
need to be overridden. But if the user needed to, they could always just
override the `widget()` method as a whole in the traditional manner.
> >
>
> That makes sense and aligns with my experiences using a similar
approach. I'd love to get a working version of this as a patch and get
some more eyes on it.
>
A test of the usefulness of the bootstrap would be to apply it to the core
widgets and see how much duplication is removed. Working on that might
also show other parts that could be abstracted.
Also, IMO, the patch is hard to follow because there are so many
extraneous whitespace changes in it. Maybe you could strip it down to just
the functional code changes, and it would be easier to work with?
--
Ticket URL: <https://core.trac.wordpress.org/ticket/32470#comment:12>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list