Skip to content

Use remote up/down key for previous/next scene #762

Merged
damontecres merged 8 commits into
damontecres:mainfrom
jbatesy:holdup
May 8, 2026
Merged

Use remote up/down key for previous/next scene #762
damontecres merged 8 commits into
damontecres:mainfrom
jbatesy:holdup

Conversation

@jbatesy

@jbatesy jbatesy commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

Resolves #760

This pull request enhances the playback navigation experience by enabling Up/Down key navigation for both scene and marker playlists, and introduces "hold to skip" functionality for these keys. It also updates the relevant preference descriptions to reflect these changes.

Playback navigation improvements:

  • Added support for using the Up/Down keys for previous/next navigation in both scene and marker playlists, not just marker playlists. (PlaybackPageContent.kt, StashPreference.kt, strings.xml) [1] [2] [3]
  • Implemented "hold to skip" functionality: holding Up or Down triggers immediate previous/next media item navigation, while a short press only shows the player controls. (PlaybackPageContent.kt) [1] [2]

User interface and preferences:

  • Updated the preference summary and string resources to clarify that Up/Down navigation applies to both scene and marker playlists. (StashPreference.kt, strings.xml) [1] [2].

@jbatesy

jbatesy commented Feb 11, 2026

Copy link
Copy Markdown
Contributor Author

@damontecres This is ready for review when you have the time.

@damontecres damontecres left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

It would be better to use the built-in KeyEvent repeat count. Each repeat is roughly 250-300ms, so you could wait until repeatCount==2.

@jbatesy

jbatesy commented Mar 14, 2026

Copy link
Copy Markdown
Contributor Author

Haven't tested this in depth, will update you when it's ready for another look.

@jbatesy

jbatesy commented Apr 16, 2026

Copy link
Copy Markdown
Contributor Author

@damontecres I cherry picked the commits in my branch over the upcoming release branch (locally) & it works well, changes also work on main branch. Please review when you have the time.

@jbatesy jbatesy requested a review from damontecres May 7, 2026 16:45

@damontecres damontecres left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks good to me!

@damontecres damontecres merged commit 9ebc400 into damontecres:main May 8, 2026
2 checks 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.

[FEA] - Use remote up/down key for previous/next scene

2 participants