Stop using pass-through refs and instead use forwardRef

[t3chguy]

Why?

The forwardRef function is React’s standard way to expose DOM nodes from components to their parents. Using this spares us having to define custom reference parameters in property interfaces.

Reference: React docs: https://beta.reactjs.org/reference/react/forwardRef

What?

We’ll do away with any custom pass-through ref parameters such as inputRef or innerRef and replace them with forwardRef calls.

Plan

This can be split into tasks consisting of each component which accepts an inputRef or innerRef (name doesn't matter) ref pass-through - involves updating it to use forwardRef and updating all consumers of that Component to use ref.

Tasks

Beta Give feedback

1. AccessibleButton
2. AutoHideScrollbar
3. StyledCheckbox
4. StyledRadioButton
5. Field
6. Dropdown
7. SpacePanel
8. SpaceTreeLevel

Internal references

myhours: https://app.myhours.com/#/projects/1975122/overview

[t3chguy]

Due to typing restrictions this isn't really feasible, forwardRef asserts Ref which can be RefCallback or RefObject, most of our usages require RefObject so we will need to maintain this non-standard behaviour