Summary: | Scale widget does not implement WIDGETCLASS parameter | ||
---|---|---|---|
Product: | Chameleon | Reporter: | Bart van den Eijnden <bartvde@osgis.nl> |
Component: | Widget | Assignee: | Bart van den Eijnden <bartvde@osgis.nl> |
Status: | RESOLVED FIXED | ||
Severity: | enhancement | CC: | jmckenna@gatewaygeomatics.com |
Priority: | P2 | ||
Version: | 1.99 | ||
Target Milestone: | --- | ||
Hardware: | PC | ||
OS: | Windows 2000 | ||
Whiteboard: |
Agreed on the widget class. Also, it should implement WIDGETSTYLE as per other widgets. Please proceed and implement both at your leisure. Re the keypress event, I like this idea but I don't want this to be something that will always be there. Please implement as a new attribute (SUBMITONKEYPRESS="[true|false]" (name?), but the default can be true. Instead of document.forms[0], please use {$this->mszHTMLForm} because widgets can now be part of any arbitrary form on a page ... use of document.forms[0] is deprecated and in fact dangerous, and needs to be removed from any existing code that still uses it (and there is some, I'm sure). Are there any other widgets that could benefit from this handy piece of code?
*** Bug 30 has been marked as a duplicate of this bug. ***
Code is now in CVS (revision 1.3 of Scale.widget.php).