-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
Apple TV: Enable long presses on TV remote; dev menu on TV device; example code #15221
Conversation
React/Base/RCTTVRemoteHandler.m
Outdated
{ | ||
[self sendAppleTVEvent:@"longPlayPause" toView:r.view]; | ||
// If shake to show is enabled on device, use long play/pause event to show dev menu | ||
[[NSNotificationCenter defaultCenter] postNotificationName:@"RCTShowDevMenuNotification" object:nil]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of using a string here, can you can change the definition of RCTShowDevMenuNotification to extern (put an RCT_EXTERN definition in the header)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
@javache has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
…ample code Summary: **Motivation** Properly support long presses on the Apple TV remote, and also enable dev menu functionality on a real Apple TV device (shaking an Apple TV doesn't work 😄 ) **Test plan** New example added to `RNTester`. Closes facebook/react-native#15221 Differential Revision: D5526463 Pulled By: javache fbshipit-source-id: a61051e86bc82a9561eefc1704bed6b1f2617e05
Motivation
Properly support long presses on the Apple TV remote, and also enable dev menu functionality on a real Apple TV device (shaking an Apple TV doesn't work 😄 )
Test plan
New example added to
RNTester
.