[Html-widget] Patch to call process in embedded widgets

Michael Gray mjg17 at eng.cam.ac.uk
Fri Aug 18 13:12:26 CEST 2006


On Fri, 18 Aug 2006, Carl Franks wrote:

> It took me a while to get through it all, but it looks really good.
> I especially like being able to now do this...
>   my $fs = $w->element( 'Fieldset', 'foo' );
>   $fs->element( 'Textfield', 'bar' );
> instead of having to use embed.

Good :-)

> There's a couple of things I noticed ( which you're probably well aware of :)
> * There's a check in HTML::Widget::element() for
> $element->{_pseudo_block}, but there doesn't seem to be any other
> mention of _pseudo_block in the code.

True.  Watch out for a refactored RadioGroup coming soon, with the guts 
split out for use in a RadioGroupFieldset or however the user wishes.  
This will illustrate a possible use for _pseudo_block.

> * There's a new $w->unwrapped. The pod for merge mentions it's use,
> but there's no use of it in the code any where.

Yes, I need to get back to that.

> I'll see if I can get the time to document the use of xhtml_strict().
> It might be a good idea for you to look what I added to the Changes
> file, to see if you agree / want to add anything else.

I'll have a look.

> It would be nice to have some tests using the new
> implicit_subcontainer(), to demonstrate it's use.

> I won't really have the time soon to make much use of these new
> changes though, so it'd be good if others can checkout the latest svn,
> and give it a whirl.
> http://oook.de/svn/trunk/HTML-Widget/

Yes Please.  Feedback welcome.  

-- 
Michael



More information about the Html-widget mailing list