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

Synced lyrics doesn't autoscroll #5487

Open
1 task done
Extarys opened this issue May 13, 2024 · 2 comments · May be fixed by #5585
Open
1 task done

Synced lyrics doesn't autoscroll #5487

Extarys opened this issue May 13, 2024 · 2 comments · May be fixed by #5585
Labels
enhancement Improve existing functionality or small fixes

Comments

@Extarys
Copy link

Extarys commented May 13, 2024

Please describe your bug

When using Jellyfin Web UI (same on desktop), the synched lyrics doesn't auto scroll.

The highlighted line should always be in the centre of the viewport IMO, or at least be visible at the bottom. See spotify (or Finamp)

(Thanks for this oh-so-awesome update, btw! <3)

Reproduction Steps

  1. Play a song with synched lyrics
  2. Click the new mega awesome button to display lyrics
  3. skip mid-way or wait until the highlighted line is below viewpoint

Jellyfin Version

10.9.0

if other:

No response

Environment

Client: Web, Desktop Linux
Browser: Firefox

Jellyfin logs

Not needed

FFmpeg logs

Not needed

Please attach any browser or client logs here

Not needed

Please attach any screenshots here

image

Code of Conduct

  • I agree to follow this project's Code of Conduct
@Extarys Extarys added the bug Something isn't working label May 13, 2024
@jellyfin-bot

This comment was marked as outdated.

@thornbill thornbill transferred this issue from jellyfin/jellyfin May 13, 2024
@thornbill thornbill changed the title [Issue]: Synched lyrics doesn't autoscroll Synced lyrics doesn't autoscroll May 13, 2024
@thornbill thornbill added enhancement Improve existing functionality or small fixes and removed bug Something isn't working labels May 13, 2024
@thornbill
Copy link
Member

This was not included as part of the initial implementation so I’m tagging it as an enhancement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improve existing functionality or small fixes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants