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: haysmark@…, doughays@…
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)

datetextbox.patch (2.8 KB) - added by Adam Peller 13 years ago.
Please review changes, including FIXME comments. Thanks.

Download all attachments as: .zip

Change History (3)

Changed 13 years ago by Adam Peller

Attachment: datetextbox.patch added

Please review changes, including FIXME comments. Thanks.

comment:1 Changed 13 years ago by Adam Peller

Summary: Suggested changes to DateTextboxSuggested changes to DateTextbox to support new _Calendar API

comment:2 Changed 13 years ago by bill

Resolution: fixed
Status: newclosed

(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.

Note: See TracTickets for help on using tickets.