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

Add new lyrics page #1054

Closed
wants to merge 5 commits into from
Closed

Add new lyrics page #1054

wants to merge 5 commits into from

Conversation

TadaHaime
Copy link

No description provided.

@felix920506
Copy link
Member

Please provide a description.

@nielsvanvelzen
Copy link
Member

Why did you close the original pull request with a pending review...

```

### Unsynchronized
Unlike syncronized lyrics, unsyncronized lyrics are easier to implament, but will be harder for users to sing along.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Unlike syncronized lyrics, unsyncronized lyrics are easier to implament, but will be harder for users to sing along.
Unlike synchronized lyrics, unsynchronized lyrics are easier to implement, but will be harder for users to sing along.

@M-Davies
Copy link
Contributor

@JustJamie4realz has this been abandoned? Happy to open a new one if needs be

@felix920506
Copy link
Member

@M-Davies just open a new PR please

M-Davies added a commit to M-Davies/jellyfin.org that referenced this pull request Sep 25, 2024
* Inspired by @JustJamie4realz work -> jellyfin#1054

Signed-off-by: Morgan Davies <[email protected]>
@M-Davies
Copy link
Contributor

@M-Davies just open a new PR please

Opened #1133

@felix920506
Copy link
Member

Closing in favor of #1133 since this thread looks abandoned

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.

5 participants