Re: QuickForm Patch
| From: | Alexey Borzov | Date: | Sun, 20 Apr 2003 17:48:30 +0000 |
| Subject: | Re: QuickForm Patch | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-15261@lists.php.net to get a copy of this message | ||
Greetings!
Jon Wood wrote:
After making a complete mess of submitting this the other day, I thought I'd submit it again. This adds a setElementDescription method to the main QuickForm class to add some instructional text to an element, and adds setValue to the static elements so they can be swapped with other elements.Well, while I see that this feature can be useful sometimes, I must say that I do not agree with its implementation. These 'descriptions' are just special cases of element 'labels' and adding new infrastructure to display them that is completely *incompatible* with labels is a bad idea. Maybe just add a possibility for labels to be arrays and when they are arrays assign them to {label1}, {label2}, ... {labelN} placeholders?