new Date("..string") example code does not work in all browser (Safari)

[jwgmeligmeyling]

new Date("..string") example code does not work in all browser (Safari)

[jwgmeligmeyling]

Not reproducible in Safari version 7.0.1 (9537.73.11) / OS X 10.9.1, please clarify your issue and the ticket might be reopened

[rsdickerson]

I'm experiencing this issue. When I view the calendar on a Windows desktop browser (Opera or Chrome) I see the events. When I view the calendar on Macbook Pro Chrome I see the events, but not viewing calendar with Safari. See the same issue when viewing calendar on iPhone using Safari (haven't tested yet with Chrome on iPhone, but expect it will work).

Since you are not able to reproduce, this issue must be caused by something I'm doing. I'll keep working on finding a resolution locally, but sure would appreciate any suggestions on what to look for.

[jwgmeligmeyling]

If we have a plunkr or jsfiddle we can test it on different browsers.

[rsdickerson]

I'll send plunkr link later today. Work firewall is blocking saving it. Thanks.

[rsdickerson]

OK, here's a plunker that for me works fine on Opera and Chrome, but on Safari the events do not appear. I'm using desktop browsers to test this, but am seeing the same issue on iPhone Safari.

http://plnkr.co/edit/47ZXOaX2ZOkndNd1q27f?p=preview

My first time using Plunker, so let me know if I need to do something different.

[jwgmeligmeyling]

Confirmed.

[rsdickerson]

The problem turns out to be how I was creating the begin and end dates (e.g. new Date("2015-06-16 07:00")) in the events. This SO question (http://stackoverflow.com/questions/6427204/date-parsing-in-javascript-is-different-between-safari-and-chrome) led me to the solution.

I changed to new Date(year,month,day,hour,minute) and now it works fine in Safari, too.

[rsdickerson]

You might want to change the demo code to use this constructor instead of the new Date(String date) just so others who follow the demo will not run into this same problem.

Thanks. Awesome product by the way. Really appreciate your efforts.

[jwgmeligmeyling]

Yes we should update the example. Luckily this isn't a bug in the calendar 😄 Thanks for your effort to sort this out!

[luigi37]

Updated demo example according to rsdickerson comment