Opened 13 years ago
Closed 13 years ago
#2999 closed defect (fixed)
Suggested changes to DateTextbox to support new _Calendar API
| Reported by: | Adam Peller | Owned by: | haysmark |
|---|---|---|---|
| Priority: | high | Milestone: | 0.9beta |
| Component: | Widgets | Version: | 0.9 |
| Keywords: | Cc: | [email protected]…, [email protected]… | |
| Blocked By: | Blocking: |
Description
I changed the onValueChanged method so that it only fires when the value is different than it previously was; this broke the date textbox -- if you selected the same date, it would not trigger. So, I created a new function called onValueSelected. I suggest referencing it via dojo.connect instead of overriding.
Patch attached.
Attachments (1)
Change History (3)
Changed 13 years ago by
| Attachment: | datetextbox.patch added |
|---|
comment:1 Changed 13 years ago by
| Summary: | Suggested changes to DateTextbox → Suggested changes to DateTextbox to support new _Calendar API |
|---|
comment:2 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
(In [8618]) (proxy checkin for Mark Hays, IBM, CCLA on file) Fixes #2935, #2938, #2962, #2992, #2936, #2937, #2977, #2981, #2999, #3000, #3004, #2812 Adds new popup code, only one Calendar for all DateTextboxes?, only one menu for AutoCompleters/Selects?. DateTextbox? works with new dijit._Calendar. DateTextbox? carries over min/max to the Calendar. All files much more consistent with the Dojo style guidelines.
Please review changes, including FIXME comments. Thanks.