[dev] Re: [cvs] commit: nag list.php
Chuck Hagenbuch
chuck at horde.org
Mon Feb 23 09:23:14 PST 2004
Quoting "Jason M. Felice" <jfelice at cronosys.com>:
> I modified Horde_UI_Widget:: to use Horde_Form_Vars:: so that the widget
> can manage it's own variables (when Horde_UI_Table:: comes to town, it
> will need to use all sorts of variables to set sort order, hide and show
> columns, determine page size and export format--these should all be
> opaque to the caller).
>
> Perhaps Horde_Form_Vars:: is overkill, or perhaps we can break it out of
> Horde/Form.php and rename it. Suggestions?
Definitely splitting out Horde_Form_Vars, or something like it.
I'd also like to see the Horde_UI stuff be as light as possible - not
extending/including base classes unless we're really getting something out of
it, etc.
-chuck
--
"Regard my poor demoralized mule!" - Juan Valdez
More information about the dev
mailing list