Opened 13 years ago
Last modified 7 years ago
#9973 open enhancement
[patch] [ccla] Select: does not allow empty value options
Reported by: | jake richardson | Owned by: | |
---|---|---|---|
Priority: | high | Milestone: | 2.0 |
Component: | Dijit - Form | Version: | 1.3.2 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description (last modified by )
Empty values for options are automatically processed as MenuSeparators, causing empty valued options being invalid.
Attachments (1)
Change History (13)
Changed 13 years ago by
Attachment: | DropDownSelect-explicit-separator-type added |
---|
comment:1 Changed 13 years ago by
Component: | General → Dijit |
---|---|
Owner: | changed from anonymous to Nathan Toone |
Summary: | DropDownSelect does not allow empty value options → [patch] [ccla] DropDownSelect does not allow empty value options |
Looks like you are covered by Sun's CCLA (you work there)?
Nathan, what do you think? I guess there could be a case where someone wants a blank choice.
comment:2 Changed 13 years ago by
Milestone: | tbd → 1.5 |
---|
comment:3 Changed 12 years ago by
Milestone: | 1.5 → future |
---|---|
Owner: | Nathan Toone deleted |
Moving my tickets to future, as I am not currently working on them.
comment:4 Changed 12 years ago by
Summary: | [patch] [ccla] DropDownSelect does not allow empty value options → [patch] [ccla] Select: does not allow empty value options |
---|
So this patch apparently changes the behavior of Select so that <option/>
shows up as an (blank) MenuItem, rather than a MenuSeparator. That's a break in backwards compatibility, although maybe it's unavoidable if we want to support <option value="">
.
Actually, the current code recognizes both <option type="separator">
and <option>
as a separator. Although on http://docs.dojocampus.org/dijit/form/Select only the former is documented, so maybe dropping the latter is not such a bad change.
See also #11545.
comment:5 Changed 11 years ago by
Owner: | set to Douglas Hays |
---|
comment:6 Changed 11 years ago by
Component: | Dijit → Dijit - Form |
---|---|
Description: | modified (diff) |
Keywords: | DropDownSelect removed |
comment:7 Changed 9 years ago by
Is there any chance of this patch being accepted and commited?
This feature would help with my client requirements and remove the problem I have with #16555
If there is some problem with backwards compatibility could this be resolved by some new property such as allowBlank?
comment:8 Changed 9 years ago by
Milestone: | future → 2.0 |
---|---|
Type: | defect → enhancement |
Given the break in backwards compatibility, this probably makes sense for 2.0.
comment:9 Changed 9 years ago by
I'm happy just with the knowledge that the change is planned. My workarounds are enough for me for now. Thanks guys.
comment:10 Changed 8 years ago by
Owner: | Douglas Hays deleted |
---|---|
Status: | new → assigned |
comment:11 Changed 8 years ago by
Status: | assigned → open |
---|
Patch that requires options to be explicitly set as a separators