IE compatibility? about datepicker

[porpoisepower]

I'm trying to adapt this to work with IE (since I have this problem with IE 11.)

So this is a post in process, as I work through issues. Hopefully this will be helpful to the authors of this package.

Lines 42 and 163 uses 'deconstructed object assignment' which IE doesn't support ( I had the same issue with IE 11) I replaced the line with individual assignments (
for example I replace 42 with

const startDate = options.startDate
const selectedDate = options.selectedDate)

Line 136 uses e => {...} instead of function(e){...} => is shorthand, (there's some other differences with this, but I didn't see it used in the function, so I think it'll be okay to the regular version)

[qodesmith]

Hey @porpoisepower, sorry for the delayed response. You should be using the datepicker.min.js file as it has already been transpiled down to ES5 and will be compatible with IE. Let me know if that works.

[noseboy]

Hi, I'm using it on IE8 and IE11 - datepicker.min.js doesn't seem to work for these. Works fine with Chrome etc.

…

On 25 October 2017 at 14:20, Aaron Cordova ***@***.***> wrote: Hey @porpoisepower <https://github.com/porpoisepower>, sorry for the delayed response. You should be using the datepicker.min.js file as it has already been transpiled down to ES5 and will be compatible with IE. Let me know if that works. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#19 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACH4kaDdSu0gWcGcw6WLlbleNclgWQvtks5svzWIgaJpZM4PhDEY> .

[qodesmith]

@noseboy Datepicker is using a mutation observer and it looks like that's incompatible with IE < 11. But can you tell me what error you're getting with IE 11 specifically?

[noseboy]

At the line if (q.includes(t) throw new Error.... the console tells me that 'Object doesn't support property or method 'includes' at line 1 column 1615

…

On 25 October 2017 at 17:43, Aaron Cordova ***@***.***> wrote: @noseboy <https://github.com/noseboy> Datepicker is using a mutation observer <https://developer.mozilla.org/en-US/docs/Web/API/MutationObserver> and it looks like that's incompatible <http://caniuse.com/#search=mutationObser> with IE < 11. But can you tell me what error you're getting with IE 11 specifically? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#19 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACH4kWLuxIRd3i7Je6oGdTNqIxGSKXcjks5sv2U8gaJpZM4PhDEY> .

[qodesmith]

@noseboy Ahhh, I see. Array.includes isn't supported in IE, only in Edge. Let me make some changes to the code.

[qodesmith]

@noseboy @porpoisepower Ok my friends. I've revamped the code towards being more IE friendly. There's no more mutation observer and it now includes a very small polyfill for Array.includes. I'd love to know how things work for you guys and if there's still IE errors. As always, please use the datepicker.min.js file.

[qodesmith]

Closing this issue. If there's still IE troubles, don't hesitate to open another ticket.