Changeset 13752 for dijit

Timestamp:

05/16/08 16:35:39 (6 months ago)

Author:

doughays

Message:

Fixes #6215 !strict. Added name attribute to Button template. Moved scrollIntoVIew call from _Widget to _FormWidget to keep multiple widgets from trying to scroll at the same time.

Location:

dijit/trunk

Files:

• form/templates/Button.html (modified) (1 diff)
• form/templates/DropDownButton.html (modified) (1 diff)
• form/_FormWidget.js (modified) (1 diff)
• tests/form/Form.html (modified) (1 diff)
• _Widget.js (modified) (1 diff)

dijit/trunk/form/templates/Button.html

@@ -3 +3 @@
         waiRole="presentation"
         ><button class="dijitReset dijitStretch dijitButtonNode dijitButtonContents" dojoAttachPoint="focusNode,titleNode"
-                type="${type}" waiRole="button" waiState="labelledby-${id}_label"
+                name="${name}" type="${type}" waiRole="button" waiState="labelledby-${id}_label"
                 ><span class="dijitReset dijitInline ${iconClass}" dojoAttachPoint="iconNode" 
                         ><span class="dijitReset dijitToggleButtonIconChar">&#10003;</span 

dijit/trunk/form/templates/DropDownButton.html

@@ -3 +3 @@
         waiRole="presentation"
         ><div class='dijitReset dijitRight' waiRole="presentation"
-        ><button class="dijitReset dijitStretch dijitButtonNode dijitButtonContents" type="${type}"
+        ><button class="dijitReset dijitStretch dijitButtonNode dijitButtonContents" type="${type}" name="${name}"
                 dojoAttachPoint="focusNode,titleNode" waiRole="button" waiState="haspopup-true,labelledby-${id}_label"
                 ><div class="dijitReset dijitInline ${iconClass}" dojoAttachPoint="iconNode" waiRole="presentation"></div

dijit/trunk/form/_FormWidget.js

@@ -93 +93 @@
 
 
+        _onFocus: function(e){
+                dijit.scrollIntoView(this.domNode);
+                this.inherited(arguments);
+        },
+
         _onMouse : function(/*Event*/ event){
                 // summary:

dijit/trunk/tests/form/Form.html

@@ -361 +361 @@
                 </table>
 
-                <button dojoType=dijit.form.Button onClick="getValues();">Get Values from form!</button>
-                <button dojoType=dijit.form.Button onClick="setValues();">Set Values to form!</button>
-                <button dojoType=dijit.form.Button type="submit">Submit</button>
-                <button dojoType=dijit.form.Button type="reset">Reset</button>
+                <button dojoType="dijit.form.Button" onClick="getValues();">Get Values from form!</button>
+                <button dojoType="dijit.form.Button" onClick="setValues();">Set Values to form!</button>
+                <button dojoType="dijit.form.Button" type="submit" name="submitButton" value="Submit">Submit</button>
+                <button dojoType="dijit.form.Button" type="reset">Reset</button>
         </form>
 

dijit/trunk/_Widget.js

@@ -298 +298 @@
 
         _onFocus: function(e){
-                dijit.scrollIntoView(this.domNode);
                 this.onFocus();
         },