[HOLD][$250] Migrate ArrowKeyFocusManager.js to function component

[marcaaron]

Class Component Migration

Filenames

• src/components/ArrowKeyFocusManager.js
  • type: React.Component
  • has state values: false
  • has refs: false
  • has context: false
  • lifecycle methods: componentDidMount,componentWillUnmount

Task

• We currently have some class components in our codebase that we would like to refactor to a function component.
• Here's a link with some general advice on how to refactor a class component to a function component: https://react.dev/reference/react/Component#alternatives
• If you need additional guidance, please ask in #expensify-open-source
• Test for any regressions and verify that there are no breaking changes

[MelvinBot]

Whoops! This issue is 2 days overdue. Let's get this updated quick!

[MelvinBot]

Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[MelvinBot]

Eep! 4 days overdue now. Issues have feelings too...

[melvin-bot]

Now this issue is 8 days overdue. Are you sure this should be a Daily? Feel free to change it!

[melvin-bot]

10 days overdue. Is anyone even seeing these? Hello?

[melvin-bot]

12 days overdue now... This issue's end is nigh!

[szebniok]

Hey, I'll work on this

[roryabraham]

I created a useArrowKeyFocusManager hook that accomplishes the same thing as this component without including anything additional in the component hierarchy. I think it's a bit simpler to use. Maybe we should just migrate to using that rather than refactoring ArrowKeyFocusManager to a functional component.

[vdem0115]

I'd like to work on this issue

[olexyt]

I can work on this issue.

[roryabraham]

I don't really agree that we should do this. I think we should just use the useArrowKeyFocusManager hook instead.

[rayane-djouah]

Can I work on this issue?

[mkhutornyi]

I'd like to work on this.

[ghost]

dibs?

[dexexpert]

I hope to work on this task.

[roryabraham]

Going to close this. As stated above, this component shouldn't really be necessary in a world of functional components since we have a hook to accomplish the same thing more simply.