[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