Opened 14 years ago
Closed 14 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 14 years ago by
Attachment: | datetextbox.patch added |
---|
comment:1 Changed 14 years ago by
Summary: | Suggested changes to DateTextbox → Suggested changes to DateTextbox to support new _Calendar API |
---|
comment:2 Changed 14 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.