[HOLD App Issue #25125] [TS migration] Migrate 'ArrowKeyFocusManager.js' component to TypeScript

[melvin-bot]

TypeScript migration

Make sure you read through our TypeScript's style guide, cheatsheet and PropTypes conversion table before you start working on this migration issue.

Files

Path	Dependencies
src/components/ArrowKeyFocusManager.js	4

[melvin-bot]

This issue has not been updated in over 15 days. eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

[melvin-bot]

@melvin-bot[bot], this Monthly task hasn't been acted upon in 6 weeks; closing.

If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.

[fabioh8010]

⚠️ Note

This component should be removed and its usage replaced with useArrowKeyFocusManager hook!

[melvin-bot]

@melvin-bot[bot], this Monthly task hasn't been acted upon in 6 weeks; closing.

If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.

[filip-solecki]

It probably should not be closed
cc @mountiny

[fabioh8010]

@filip-solecki Do you want to lead this replacement of ArrowKeyFocusManager to useArrowKeyFocusManager? 🙂

@hayata-suenaga Could you reopen this issue? Thanks!

[filip-solecki]

Yes, I'll do it, but I am blocked by OptionsSelector as it is not migrated to functional component so I can't use any hook there. I am currently working on removing OptionsSelector from our codebase but it will take some time to replace it with SelectionList as it needs some improvements

[fabioh8010]

@filip-solecki Could you link OptionsSelector issue here?

[filip-solecki]

Sure, here is the blocker

[fabioh8010]

Thanks!

@hayata-suenaga Could we also put this issue on HOLD because of #25125? Thanks!

[mountiny]

Closing per here