-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Method isSelected(date) missing #3
Comments
Good idea! I can add it. And maybe some more isXY(..) methods would be nice, like: calendar.isVisible(date); // Returns true if date is displayed
calendar.isInMonth(date); // Returns true if date is in active month BTW, I personaly prefer |
Yes, that was the idea. I prefer isSelected too (isSelect is a bad grammatical error, sorry). |
Implemented and are now live on developer version v1.4-beta for testing
Also, to be honest, when I implemented the first release, I just added the select methods as a simple minimal feature. So, some more features may be needed, like |
Hi mate, great job you're doing. |
Thanks. As this is a different issue, I opend one for it here #4. |
The method Examples : myCalendar.getSelected();
// Returns [1483567200000, 1483394400000, 1483308000000, 1483221600000]
myCalendar.getSelected({sort:true});
// Returns [1483221600000, 1483308000000, 1483394400000, 1483567200000]
myCalendar.getSelected({sort:"desc",type:"DD-MM-YYYY"});
// Returns ["05-01-2017", "03-01-2017", "02-01-2017", "01-01-2017"]
myCalendar.getSelected({sort:"asc",type:"date"});
// Returns [Sun Jan 01 2017 00:00:00 GMT+0200 (GTB Standard Time), Mon Jan 02 2017 00:00:00 GMT+0200 (GTB Standard Time), Tue Jan 03 2017 00:00:00 GMT+0200 (GTB Standard Time), Thu Jan 05 2017 00:00:00 GMT+0200 (GTB Standard Time)] |
This calendar is good and provides all the functions someone can need, good job!
One enhancement that I could suggest is to implement a method to check if a certain date is already selected.
The text was updated successfully, but these errors were encountered: