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

chore!: drop support for Internet Explorer #109

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

jonkoops
Copy link
Collaborator

@jonkoops jonkoops commented Nov 9, 2024

What kind of change does this PR introduce?

  • Bug Fix
  • Feature
  • Refactoring
  • Style
  • Build
  • Chore
  • Documentation
  • CI

Did you add tests for your changes?

  • Yes, my code is well tested
  • Not relevant

If relevant, did you update the documentation?

  • Yes, I've updated the documentation
  • Not relevant

Summary
Removes conditional code and tests for Internet Explorer and drops support for it.

Does this PR introduce a breaking change?
Yes, it removes support for Internet Explorer entirely.

Other information
Not relevant.

@jonkoops jonkoops changed the title chore:! drop support for Internet Explorer chore!: drop support for Internet Explorer Nov 9, 2024
@coveralls
Copy link

coveralls commented Nov 9, 2024

Pull Request Test Coverage Report for Build 11756318444

Details

  • 9 of 9 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 11756009658: 0.0%
Covered Lines: 84
Relevant Lines: 84

💛 - Coveralls

Signed-off-by: Jon Koops <[email protected]>

BREAKING CHANGE: drops support for Internet Explorer
@jonkoops
Copy link
Collaborator Author

jonkoops commented Nov 9, 2024

@rolandjitsu this one is ready to go, can I ask you for a review?

@jonkoops jonkoops requested a review from rolandjitsu November 9, 2024 17:36
@rolandjitsu
Copy link
Collaborator

There's apparently an IE mode in Edge that's supported until 2029 (see internet explorer 11 has retired)

And of course, we have committed to supporting IE mode in Microsoft Edge through at least 2029.

Not sure what that translates to. There may also be clients out there that are still using IE 🤔

I just thought that's worth mentioning it.

Copy link
Collaborator

@rolandjitsu rolandjitsu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It look ok. I just wonder if we should just drop support for IE11 🤔

@jonkoops
Copy link
Collaborator Author

We're already shipping syntax that is no longer supported in Internet Explorer, so this realistically won't be an issue. Also the compatibility mode in Edge is intended to be used only on an allow-list basis for sites that experience compatibility issues, not as a blanket feature to enable on all sites, as that would most certainly cause most sites not to function.

@jonkoops
Copy link
Collaborator Author

@rolandjitsu any objection against merging this one? I have a couple of PRs that will build off this to resolve the remaining linting issues.

@rolandjitsu rolandjitsu merged commit 34575db into react-dropzone:beta Nov 13, 2024
6 checks passed
Copy link

🎉 This PR is included in version 3.0.0-beta.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@jonkoops jonkoops deleted the rm-ie-support branch November 13, 2024 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants