Add tvOS Siri Remote navigation - #128
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit e056e02. Configure here.
| else { return } | ||
|
|
||
| selectedIndex = index | ||
| } |
There was a problem hiding this comment.
Remote navigation double-steps selection
High Severity
onMoveCommand always calls move(by:) even when a result row already has focus, while those rows are also .focusable() and onChange(of: focusedResultID) updates selectedIndex. One Siri Remote swipe can advance the highlight twice—focus moves to the neighbor and move(by:) steps again—so browsing results overshoots. The comment notes moves are meant to nudge selection when focus is on the search field, but that condition is not enforced.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit e056e02. Configure here.


Summary
onMoveCommand/ Play-Pause / Menu on tvOSselectedIndexTest plan
swift testCloses #120.
Made with Cursor