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

Feat: Add onLongSelect #156

Merged
merged 9 commits into from
Sep 18, 2024
Merged

Feat: Add onLongSelect #156

merged 9 commits into from
Sep 18, 2024

Conversation

remilry
Copy link
Contributor

@remilry remilry commented Sep 18, 2024

Solves #36

Web

Enregistrement.de.l.ecran.2024-09-18.a.10.56.30.mov

AppleTV

Enregistrement.de.l.ecran.2024-09-18.a.11.00.38.mov

AndroidTV

Enregistrement.de.l.ecran.2024-09-18.a.11.09.25.mov

}[evt.eventType];

if (!mappedKey) {
return;
}

// We only want to handle keydown fon long select
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"because it delays input and we don't want to add lag on all inputs"

(is that correct?)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is because we don't want to handle twice the long select (when keydown and when keyup)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will refine the comment

@@ -6,7 +6,6 @@ import {
SpatialNavigationVirtualizedListRef,
} from 'react-tv-space-navigation';
import { Page } from '../components/Page';
import '../components/configureRemoteControl';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wtf 😂 thanks for fixing it!

Copy link
Member

@pierpo pierpo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! We finally have it 😁

@remilry remilry merged commit 5c4913e into main Sep 18, 2024
1 check passed
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.

2 participants