Opened 12 years ago
Closed 12 years ago
#9317 closed enhancement (fixed)
Add isSelectedDate to dijit._Calendar
Reported by: | ryangroth5 | Owned by: | bill |
---|---|---|---|
Priority: | high | Milestone: | 1.4 |
Component: | Dijit | Version: | 1.3.0 |
Keywords: | date select calendar | Cc: | [email protected]… |
Blocked By: | Blocking: |
Description
There is a basic assumption that the _Calendar has one selected date which limits the ability to select multiple dates on the _Calendar. While this might be an uncommon case for the dijit.* components, it is an requirement to facilitate with the addition of one method.
Please review the following patch and if it doesn't seem to violate any design principles the additional capability that subclasses could provide would be beneficial to the community. I actually think that it improves the readability of the code as well.
I have a CLA on file if you want to incorporate the patch as is.
Attachments (1)
Change History (3)
Changed 12 years ago by
Attachment: | dijit.diff added |
---|
comment:1 Changed 12 years ago by
Owner: | set to bill |
---|---|
Status: | new → assigned |
OK, I'll add it but as _isSelectedDate since it's not a typical extension function (ie, it only makes sense if you are subclassing Calendar, it doesn't make sense to just define that one method).
comment:2 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
adds isSelectedDate to _Calendar