Skip to content

refactor: upgrade react-transition-group v2 to v4 and migrate string refs to createRef - #2840

Open
ankita10119 wants to merge 1 commit into
v13from
SDK-10306-1
Open

refactor: upgrade react-transition-group v2 to v4 and migrate string refs to createRef#2840
ankita10119 wants to merge 1 commit into
v13from
SDK-10306-1

Conversation

@ankita10119

Copy link
Copy Markdown
Contributor

Changes

Upgrades react-transition-group from ^2.2.1 to ^4.4.5. RTG v4 removes internal use of the deprecated ReactDOM.findDOMNode, requiring an explicit nodeRef prop on CSSTransition. This in turn drove a full migration from legacy string refs (this.refs.*) and ReactDOM.findDOMNode calls to React.createRef() across the UI layer.

Components changed:

  • ui/box/chrome.jsx - CSSTransition gains nodeRef props; screen, auxiliary, and globalMessage refs converted to createRef
  • ui/box/container.jsx - this.refs.chromethis.chromeRef
  • ui/box/header.jsx - getDOMNode uses this.headerRef.current instead of findDOMNode(this)
  • ui/box/multisize_slide.js - this.refs[key] replaced with a childRefs map of createRef instances
  • ui/list.jsx - this.refs.highlighted and findDOMNode removed
  • All 8 input components (captcha, email, mfa_code, password, phone_number, text, username, vcode) - this.refs.inputthis.inputRef.current

Also removes the unused events dev dependency.

References

Testing

  • This change adds unit test coverage
  • This change adds integration test coverage
  • This change has been tested on the latest version of the platform/language

All 401 existing tests pass. Visual/animation behaviour should be verified manually, particularly the slide transition between screens and the auxiliary pane animation.

Checklist

@ankita10119
ankita10119 requested a review from a team as a code owner July 21, 2026 17:54
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.

1 participant