On Wed, Mar 08, 2006 at 09:54:04PM -0000, Michele Cella wrote:
> 
> Jason Chu wrote:
> >
> > For those that aren't following the ticket, I uploaded a newer version of
> > the patch that follows the r913 way of doing things.
> >
> 
> Committed, thanks Jason! ;-)
> 
> Ciao
> Michele
> 
> PS
> Have you found the new way of doing things better?

I think it makes things simpler, yes.  If there was an easy way to get a
widget by name it would be nicer though, except your named-widget .display
lines would get really ugly:

field_by_name("blah").display(value_for(field_by_name("blah"),
**options_for(field_by_name("blah"))

You'd almost want a different display method, that passed all that stuff by
default..

form_display(field_by_name("blah"))

And then you could pass overriding variables inside the form_display
method...  I don't know if we want something like that though.

Jason

-- 
If you understand, things are just as they are.  If you do not understand,
things are just as they are.

Attachment: pgpaCQHAavcHK.pgp
Description: PGP signature

Reply via email to