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

Consider dropping support for React versions <= v16 #1041

Open
doeg opened this issue Feb 16, 2024 · 3 comments
Open

Consider dropping support for React versions <= v16 #1041

doeg opened this issue Feb 16, 2024 · 3 comments

Comments

@doeg
Copy link

doeg commented Feb 16, 2024

Currently, the package.json file notes the following React peer dependencies:

  "peerDependencies": {
    "react": "^0.14.0 || ^15.0.0 || ^16 || ^17 || ^18",
    "react-dom": "^0.14.0 || ^15.0.0 || ^16 || ^17 || ^18"
  },

v15 and v16 were released in 2017 (seven years ago) and v0.14.0 is from 2015 (nine years ago).

It seems fair to drop at least v0.14.0 and v15 in the next major release for react-modal. (To be explicit at the risk of being pedantic, updating peer dependencies is generally considered a breaking change... even when the peer dependencies are ancient.) 😃

Dropping support for React v16 seems reasonable too, though it may be nice to support that one through v4 if we feel there are time/resources to do so.

Note that #1019 raises some issues with react-modal compatibility with React v18. These are particularly tricky as it'll require some care to ensure the fixes work for v17.

@diasbruno
Copy link
Collaborator

I'm against dropping support if it's not necessary. But I also don't have a plan right now.

Maybe, first, we need to understand what's going on with #1090 to see if we need to really make that breaking change.

From what I can see, it's just a matter of adding startTransition on the users' side (but I might be wrong - radiohead 🎵).

@diasbruno
Copy link
Collaborator

But I also wouldn't oppose if we can speculate on the adoption of versions 16.3+ (yep, exactly 16.3 up)

@diasbruno
Copy link
Collaborator

diasbruno commented Feb 26, 2024

I think I'm being super defensive about this.

Maybe, the obsolete class component is dragging this project down.

It'd be nice to set a millestone for this changes as we have more people working on this project.

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

No branches or pull requests

2 participants