Skip to content
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

Replace Promise with async #191

Conversation

claudiodekker
Copy link
Member

@claudiodekker claudiodekker commented Aug 19, 2020

Minor simplification for #116

@reinink reinink merged commit 5b9eab2 into inertiajs:scroll-region-improvements Aug 19, 2020
@reinink
Copy link
Member

reinink commented Aug 19, 2020

Love it.

@reinink
Copy link
Member

reinink commented Aug 19, 2020

One question, will this work with IE11? 😬

@claudiodekker
Copy link
Member Author

claudiodekker commented Aug 19, 2020

Oof, yeah, no, it won't. Completely forgot about IE11 🙀🙃 My bad!

@reinink
Copy link
Member

reinink commented Aug 19, 2020

I’m seriously considering dropping IE11 support. 🤔

@pedroborges
Copy link
Collaborator

Microsoft itself is dropping IE11 support for their services.

@claudiodekker
Copy link
Member Author

claudiodekker commented Aug 19, 2020

Yeah, it's worth considering. I was just thinking about asking your thoughts on it as well.

Personally, I have mixed feelings about it.. Inertia is young & (unfortunately) still small enough to be able to make big decisions like this without arguably huge impact, but at the same time I can imagine some people already using it in apps built for (for example) institutions, where IE is the only option.

Microsoft just announced they're killing support for their own IE11 & Legacy Edge as early as December: https://techcommunity.microsoft.com/t5/microsoft-365-blog/microsoft-365-apps-say-farewell-to-internet-explorer-11-and/ba-p/1591666

Vue 3 will still be supporting it, but will apparently ship a separate build that has "proxied" support for it vuejs/rfcs#183 & https://medium.com/vue-mastery/evan-you-previews-vue-js-3-0-ab063dec3547

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants