Re: [PEPr] Comment on XML::XML_XUL

Stephan Schmidt wrote:

Hi,

couple of things: - too many 'empty' classes.. - why not store the schema (eg. attributes types in a 'data' file), and just use base XML_XUL_Element...

Most of these classes will get additional methods, like it is with Grid, Tree, Tabbox, etc...

It would be better to at least remove the empty ones.. for the time being.. (just stuff them in a 'if_needed') folder :)..

A few of the methods use >3 arguments.. it would be nice to simplify them,

or start using the assoc array method of setting.. Could you tell me which of the methods you are talking about?

I only found addTab on Tabbox.. - as there's so many files.. it's a bit of a chore looking through alot of them...

I would prefer $tb = XML_XUL_Element::Factory('textbox');

I need the document to create the element, as the document knows it's namespace and sets this namespace in the element...

does XUL have namespaces????? - I missed any reference to that on XUL planet... BTW - XML_XUL_INCLUDE_PATH has to go ;) I was toying with XUL today - You can make really nice interfaces with it.. http://devel.akbkhome.com/xultest/test.html Regards Alan

And please, please, please make the children/attributes public.. vars..

That's no problem... Stephan

-- Can you help out? Need Consulting Services or Know of a Job? http://www.akbkhome.com