react-transition-group icon indicating copy to clipboard operation
react-transition-group copied to clipboard

feat!: support React19 & drop React 16,17 support

Open koba04 opened this issue 11 months ago • 9 comments

This only updates CI settings and tests. react-transition-group seems to work with React 19 without any updates

koba04 avatar Dec 24 '24 10:12 koba04

someone please pay attention to this PR

ddtch avatar Jan 08 '25 23:01 ddtch

its in draft and not passing CI @ddtch what do you want us to pay attention to??

@koba04 lmk when this is updated to pass CI

jquense avatar Jan 09 '25 14:01 jquense

@jquense Yeah, CI fails are caused because of old dependencies and the Node version, and react-transition-group works with React v19 as long as we use nodeRef, so I think this doesn't block anything.

I'd say we should update our build settings , outdated dependencies and documentation system before updating React version.

koba04 avatar Jan 10 '25 08:01 koba04

you wanna do it?

jquense avatar Jan 10 '25 18:01 jquense

Yeah, I'll do it when I have time, but it would be helpful someone takes that.

The documentation also depends on PropTypes, which was removed in v19, so it would be better to update the documentation as well. (Keeping PropTypes won't have any issues, it's just ignored, so this isn't a requirement)

https://react.dev/blog/2024/04/25/react-19-upgrade-guide#removed-deprecated-react-apis

koba04 avatar Jan 11 '25 08:01 koba04

it would be so helpful if this MR would be included soon🙏

sergioviniciuss avatar Feb 05 '25 15:02 sergioviniciuss

@sergioviniciuss I think react-transition-group works with React v19, do you have any issue that this can fix?

koba04 avatar Feb 21 '25 04:02 koba04

@koba04 I'm seeing an issue when using the Transition component on React v19:

TypeError: react_dom__WEBPACK_IMPORTED_MODULE_3___default(...).findDOMNode is not a function

danielstclair avatar Feb 24 '25 18:02 danielstclair

@koba04 I'm seeing an issue when using the Transition component on React v19:

TypeError: react_dom__WEBPACK_IMPORTED_MODULE_3___default(...).findDOMNode is not a function

You need to add a ref and it'll work like this: <CSSTransition nodeRef={nodeRef} . . </CSSTransition>

This will make it not crash but it will still fail on typing through eslint, so you might want to disable it for that.

samin15 avatar Mar 07 '25 02:03 samin15