Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Building for TV Devices #1566

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

dev-seb
Copy link

@dev-seb dev-seb commented Mar 18, 2020

Add support for TV Devices:

  • Platform.isTV
  • hasTVPreferredFocus
  • nextFocusLeft
  • nextFocusRight
  • nextFocusTop
  • nextFocusBottom
  • TVEventHandler global events
  • Touchable events
  • Touchable UI changes on select

Demo app : https://github.com/dev-seb/react-native-tv-demo

The expected behaviour is the same as for AndroidTV and tvOS, and only if Platform.isTV property is true.

For Platform.isTV is should be possible to parse the User Agent, but env variable seems more suitable to specify whether or not the current app is targeting TVs.

For spatial navigation, the demo relies for web on the third party library Spatial Navigation Polyfill.

Add support for : 

 - TVEventHandler events
 - Platform.isTV 
 - hasTVPreferredFocus property
 - nextFocusUp, nextFocusRight, nextFocusDown, nextFocusLeft properties
 - back event from remote control
- Enable TV support on TouchableHighlight only and if Platform.isTV is true
- Move Key Event handler to useTVEvents module
- Support both NodeHandle and nativeID for nextFocus props
@numericgraphics
Copy link

numericgraphics commented Apr 16, 2021

Hello, possible to solve those conflicts @dev-seb ? thx

@dev-seb
Copy link
Author

dev-seb commented Apr 16, 2021

It should be solved with 549ec21

@numericgraphics
Copy link

Hi @necolas , possible to have this PR in the next release ? should the warning be addressed @dev-seb ?
thx anyway

@freemanchari
Copy link

@necolas What became of this PR? This is really needed

@svbutko
Copy link

svbutko commented Nov 24, 2023

Is there any way to help merge this PR?

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.

4 participants