Re: [PEPr] Comment on XML::XML_XUL
| From: | Stephan Schmidt | Date: | Mon, 23 Feb 2004 08:20:45 +0000 |
| Subject: | Re: [PEPr] Comment on XML::XML_XUL | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-25793@lists.php.net to get a copy of this message | ||
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...
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 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...
And please, please, please make the children/attributes public.. vars.. That's no problem...Stephan