Skip to content

Comments

Replace deprecated findDOMNode with modern ref-based approach [Claude Code Web]#1546

Draft
bobular wants to merge 1 commit intomainfrom
claude/replace-finddomnode-refs-012pmuPdoRerMqRDvsgKyUtX
Draft

Replace deprecated findDOMNode with modern ref-based approach [Claude Code Web]#1546
bobular wants to merge 1 commit intomainfrom
claude/replace-finddomnode-refs-012pmuPdoRerMqRDvsgKyUtX

Conversation

@bobular
Copy link
Member

@bobular bobular commented Nov 15, 2025

  • RecordUI.jsx: Replace findDOMNode in ref callback with createRef
  • LegacyParamController.tsx: Add container ref to replace findDOMNode(this)
  • AnswerFilter.jsx: Replace deprecated string refs and findDOMNode
  • IndeterminateCheckbox.tsx: Use ref instead of findDOMNode to set indeterminate
  • Loading.tsx: Add container ref for Spinner initialization
  • Popup.tsx: Use callback ref directly without findDOMNode
  • Sticky.jsx: Replace findDOMNode(this) with createRef
  • FieldList.jsx: Use callback ref for CheckboxTree
  • Histogram.jsx: Add container ref for chart operations

This eliminates all deprecated React.findDOMNode usages in the codebase, improving compatibility with React strict mode and future React versions.

- RecordUI.jsx: Replace findDOMNode in ref callback with createRef
- LegacyParamController.tsx: Add container ref to replace findDOMNode(this)
- AnswerFilter.jsx: Replace deprecated string refs and findDOMNode
- IndeterminateCheckbox.tsx: Use ref instead of findDOMNode to set indeterminate
- Loading.tsx: Add container ref for Spinner initialization
- Popup.tsx: Use callback ref directly without findDOMNode
- Sticky.jsx: Replace findDOMNode(this) with createRef
- FieldList.jsx: Use callback ref for CheckboxTree
- Histogram.jsx: Add container ref for chart operations

This eliminates all deprecated React.findDOMNode usages in the codebase,
improving compatibility with React strict mode and future React versions.
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.

2 participants