fix startDay and introduce safer way of adding days - Githubissues

jekor / drcal

A Minimalist JavaScript Calendar (not a date picker)

http://www.minjs.com/#drcal

MIT License

26 stars 5 forks source link

fix startDay and introduce safer way of adding days #3

Closed midzer closed 8 years ago

midzer commented 8 years ago

Hi. This patch adresses two issues i encountered while implementing drcal in a website.

In some cases a modified start day of week did not work for me as expected: Having 'startDay': 1 drcal rendered May 2016 with Monday 2nd first. So there is the whole previous week with May, 1st Sunday missing.
In rare occasions the milliseconds approach fails: I saw same day twice, on Oct 30th i think. Answers in http://stackoverflow.com/questions/563406/add-days-to-datetime helped to fix this problem.

Great calendar. It's a pleasure to work with it :)

jekor commented 8 years ago

Thanks for the patch. I really appreciate it. The changes are clean and easy to understand.