Opened 9 years ago

Closed 9 years ago

#12925 closed defect (fixed)

Change on visibility for method toggleDate in dojox.widget.MultipleSelectCalendar

Reported by: Jean-Rubin Leonard Owned by: Adam Peller
Priority: high Milestone: 1.7
Component: Dojox Version: 1.6.1
Keywords: MultipleSelectCalendar toggleDate Cc:
Blocked By: Blocking:

Description

The method toggleDate had been created as a private one. Upon use however it makes sense to allow the developpers to turn a date on / off programmatically. Further more the toggleDate method has been split into two atomic operations: selectDate and unselect Date.

Attachments (2)

MSC3.diff (5.3 KB) - added by Jean-Rubin Leonard 9 years ago.
MSC3.2.diff (7.0 KB) - added by Jean-Rubin Leonard 9 years ago.

Download all attachments as: .zip

Change History (6)

Changed 9 years ago by Jean-Rubin Leonard

Attachment: MSC3.diff added

comment:1 Changed 9 years ago by Adam Peller

Milestone: tbd1.7

comment:2 Changed 9 years ago by Adam Peller

Type: enhancementdefect

Changed 9 years ago by Jean-Rubin Leonard

Attachment: MSC3.2.diff added

comment:3 Changed 9 years ago by Jean-Rubin Leonard

Using dojo.date.stamp.fromISOString to create dates from iso string instead of a more cumbersome method that was used. Also I made some changes so that upon instantiation, the latest date with which the calendar is created becomes the currentDate so that when we reopen it, we are positionned at that date.

comment:4 Changed 9 years ago by Adam Peller

Resolution: fixed
Status: newclosed

(In [25021]) patch from simpli Fixes #12925, rename toggleDate and code cleanup \!strict

Note: See TracTickets for help on using tickets.