yariksav / vuetify-confirm

Extends vuetify.js confirm dialog
MIT License
142 stars 42 forks source link

Multiple condition if statement improvement #4

Closed ahmozkya closed 6 years ago

ahmozkya commented 6 years ago

It is quite likely that my .babelrc file is correct but multiple condition if statement throws ) error on ie11. I think it's better like this (just in case). If I'm wrong please simple discard my pull request.

Thanks for your effort.

yariksav commented 6 years ago

Hello @ahmozkya 🙌 Thank you for your pull request! 👍But I don't have ie11 in my case Did you try is your changes works correctly in ie11? Can you show here full error message from ie11?

ahmozkya commented 6 years ago

Hello @yariksav ,

Can you show here full error message from ie11?

Here is with unmodified setup; capture

Did you try is your changes works correctly in ie11?

Yes.

The issue only on ie11, edge works without any problem.

PS: I also don't have windows os. I'm trying with virtualbox. So if you need any further info let me know.

yariksav commented 6 years ago

Published ver 0.0.5 vith your changes

ahmozkya commented 6 years ago

Thanks 👍