Skip to content

Added frame by frame control by using "," and "."#7132

Open
nixietab wants to merge 1 commit intojellyfin:masterfrom
nixietab:master
Open

Added frame by frame control by using "," and "."#7132
nixietab wants to merge 1 commit intojellyfin:masterfrom
nixietab:master

Conversation

@nixietab
Copy link

@nixietab nixietab commented Sep 10, 2025

Changes
Added frame-by-frame navigation functionality to the video player.
When the video is paused, users can now advance one frame forward by pressing "." or go one frame backward by pressing ",".

Issues
Resolves #3087

This is a proof of concept on how would work

@nixietab nixietab requested a review from a team as a code owner September 10, 2025 05:56
@jellyfin-bot
Copy link
Collaborator

jellyfin-bot commented Sep 10, 2025

Cloudflare Pages deployment

Latest commit 722759f
Status ✅ Deployed!
Preview URL https://3ba959c8.jellyfin-web.pages.dev
Type 🔀 Preview

View build logs

@sonarqubecloud
Copy link

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

ESLint doesn't pass. Please fix all ESLint issues.

Comment on lines +98 to +99
document.activeElement.tagName === 'INPUT' ||
document.activeElement.tagName === 'TEXTAREA' ||
Copy link
Contributor

Choose a reason for hiding this comment

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

'||' should be placed at the beginning of the line. @stylistic/operator-linebreak

Fix available:

Suggested change
document.activeElement.tagName === 'INPUT' ||
document.activeElement.tagName === 'TEXTAREA' ||
document.activeElement.tagName === 'INPUT'
|| document.activeElement.tagName === 'TEXTAREA' ||

Comment on lines +99 to +100
document.activeElement.tagName === 'TEXTAREA' ||
document.activeElement.isContentEditable
Copy link
Contributor

Choose a reason for hiding this comment

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

'||' should be placed at the beginning of the line. @stylistic/operator-linebreak

Fix available:

Suggested change
document.activeElement.tagName === 'TEXTAREA' ||
document.activeElement.isContentEditable
document.activeElement.tagName === 'TEXTAREA'
|| document.activeElement.isContentEditable

@flying-sheep
Copy link

flying-sheep commented Nov 9, 2025

Superseded by #7315, where I use the stream’s FPS

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.

Allow per-frame playback navigation

3 participants