-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
Drop IE11 support. #1747
Drop IE11 support. #1747
Conversation
Microsoft is about to do the same thing: https://docs.microsoft.com/en-us/lifecycle/products/internet-explorer-11 Plus, Microsoft already has "IE Mode" in Edge which it will support until 2029: https://www.microsoft.com/en-us/edge/business/ie-mode There will be no reason to keep IE support.
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/docsify-core/docsify-preview/7SmaYHDGt6HWtL5y83SaCEZi7WVr |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 8b70b32:
|
As much as I appreciate the sentiment (we're all ready to put IE11 behind us), dropping support for IE11 is not as easy as removing a line item from our We have discussed and agreed to a plan for dropping support for IE11 in the past (e.g., #657 (comment)). The TL;DR is that we can do this with a major version bump (v4 => v5), but we should not publish a new major version just to drop IE11 support. The issue here is Docsify's history of linking to CDN URLs locked to the Equally important, we should not be so quick to assume legacy browsers (like IE11) are not being used or supported elsewhere. The fact that MS has and will continue to support IE11 up to June 15, 2022 indicates that there is a rational for this. Here's another data point to consider: I created css-vars-ponyfill to add support for CSS custom properties (aka "CSS variables") to legacy browsers which is downloaded ~130 million times per month from one CDN (jsDelivr) and ~500 thousand times per month from npm. Here are live stats: We can't derive the number of IE11 users from those stats, but the stats do indicate that many developers and site visitors continue to use tools designed specifically to support IE11 even into 2022 (as sad as that may be). I continue to get questions from developers to this day asking things like how to integrate the ponyfill into their app which is using the latest version of Angular. For comparison, here are the same live stats for docsify (~7M/mo for jsDelivr, 117K/mo for npm when written): Just to be clear, I am not proposing that we continue to prioritize IE11 support. My recent PR (#1743) was not intended to imply that; I just wanted v4's legacy support to be in better shape when the day comes that we finally move on to v5. If we want to drop IE11 support sooner than later, let's discuss if/how we can handle the bump to v5 in a non-breaking way first and proceed from there. FYI: The linked issues above were fixed by #1743. I've marked them as resolved and closed them. @docsifyjs/core: I vote to close this PR and drop IE11 support with the next major version. Thoughts? |
What if we move the a new version (whether it is 4.x, 5.x, or etc) build output to a new path, and leave the last IE-compatible version at the same path so that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with @jhildenbiddle that we need not drop this IE support for now.
If user has new request on IE 11 we could mention them we will remove IE support in the future.
Yep. That's an option. Users would lose the ability to load the latest version of Docsify using the But for now...
Let's close this PR and create a new issue where we can discuss approaches for releasing v5 (hopefully with a "how should we do it" focus instead of a "what should it contain" debate). @trusktr -- I generally prefer to let authors close their own issues/PRs, so I'm leaving this open until you're ready. |
Closing due to reviewer consensus and lack of inactivity. We'll revisit legacy support in v5. |
Summary
Microsoft is about to do the same thing: https://docs.microsoft.com/en-us/lifecycle/products/internet-explorer-11
Plus, Microsoft already has "IE Mode" in Edge which it will support until 2029: https://www.microsoft.com/en-us/edge/business/ie-mode
There will be no reason to keep IE support soon.
What kind of change does this PR introduce?
Docs
For any code change,
Does this PR introduce a breaking change? (check one)
If yes, please describe the impact and migration path for existing applications:
Related issue, if any:
closes #1221
closes #707
Tested in the following browsers: