Index: dojo-release-1.3.2-src/dojox/form/DropDownSelect.js =================================================================== --- dojo-release-1.3.2-src.orig/dojox/form/DropDownSelect.js 2009-09-04 11:21:11.000000000 -0700 +++ dojo-release-1.3.2-src/dojox/form/DropDownSelect.js 2009-09-04 11:22:49.000000000 -0700 @@ -62,11 +62,9 @@ // summary: // For the given option, return the menu item that should be // used to display it. This can be overridden as needed - if(!option.value){ - // We are a separator (no label set for it) + if(option.type === "separator"){ return new dijit.MenuSeparator(); }else{ - // Just a regular menu option var click = dojo.hitch(this, "_setValueAttr", option); return new dijit.MenuItem({ option: option, @@ -80,8 +78,7 @@ _addOptionItem: function(/* dojox.form.__SelectOption */ option){ // summary: // For the given option, add a option to our dropdown - // If the option doesn't have a value, then a separator is added - // in that place. + // To add a separator, set option.type = "separator". this.dropDown.addChild(this._getMenuItemForOption(option)); }, Index: dojo-release-1.3.2-src/dojox/form/_FormSelectWidget.js =================================================================== --- dojo-release-1.3.2-src.orig/dojox/form/_FormSelectWidget.js 2009-09-04 11:21:11.000000000 -0700 +++ dojo-release-1.3.2-src/dojox/form/_FormSelectWidget.js 2009-09-04 11:22:49.000000000 -0700 @@ -288,13 +288,12 @@ if(!opts){ opts = this.options = this.srcNodeRef ? dojo.query(">", this.srcNodeRef).map(function(node){ - if(node.getAttribute("type") === "separator"){ - return { value: "", label: "", selected: false, disabled: false }; - } - return { value: node.getAttribute("value"), - label: String(node.innerHTML), - selected: node.getAttribute("selected") || false, - disabled: node.getAttribute("disabled") || false }; + return { + type: node.getAttribute("type"), + value: node.getAttribute("value"), + label: String(node.innerHTML), + selected: node.getAttribute("selected") || false, + disabled: node.getAttribute("disabled") || false }; }, this) : []; } if(!this.value){