Need to add setFocusByDirection("left", "right", "top" or "bottom") about react-spatial-navigation

[asgvard]

@salvan13 Perhaps navigateByDirection is a good name. Because "navigateTo" might mean "navigate to some place" instead of direction. Also "focusNext" might mean it focuses some component.
@alfimois I'll add some comments to PR in a minute

[yarikleto]

Usually mouse wheel doesn't change focus on elements, not sure if we should add this. Can you explain your use case step by step?

Our TVs can change focusing by a wheel on the remote ;)

[salvan13]

Hi @alfimois
Usually mouse wheel doesn't change focus on elements, not sure if we should add this. Can you explain your use case step by step?

[yarikleto]

Added by that PR #36

[salvan13]

Thanks!
So basically you need a method for navigate in a certain direction programmatically. Right?

In this case I would call it navigateTo(direction) or focusNext(direction) instead of smartNavigateByDirection(direction) (since smartNavigate is not used in the public api).

what do you think @asgvard ?

[salvan13]

@alfimois also please when you open a pull request don't mix things: like refactoring + new feature.
Let's keep pull request minimal and focus on one thing.
Thanks!

[yarikleto]

Let's keep pull request minimal and focus on one thing.
@salvan13 OK, sry)

[yarikleto]

Added #36