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

Embed Synced Lyrics Into SYLT Tag #103

Open
EliasGabr opened this issue Jan 22, 2025 · 7 comments
Open

Embed Synced Lyrics Into SYLT Tag #103

EliasGabr opened this issue Jan 22, 2025 · 7 comments

Comments

@EliasGabr
Copy link

Feature Description

Hello, I would love to have an option for lyrics that is plain (no timestamps or info at the top or bottom)

Justification

The justification is that Apple Music in iPod mode doesn't support stamped lyrics, only old school ones. It would be nice as it would save a lot of time and effort removing these extra timestamps at the start and end of each line, which could be easily automated.

Additional Information

No response

@justin025
Copy link
Owner

justin025 commented Jan 22, 2025

Does the ipod work off embedded metadata or an lrc file?

@justin025
Copy link
Owner

I added the option to only download plain lyrics, you can find the exe at the bottom of the following page

https://github.com/ots-downloader/onthespot/actions/runs/12913932554

@EliasGabr
Copy link
Author

@justin025 yes it works off the metadata. It looks like this

Image
Image

Don’t get me wrong, the lyrics system on your program is really good. But sadly apple only supports real stamped lyrics for subscriptions only and not on locally stored music. Because it caps the features of phones to iPods.

I remember I had to put the lyrics metadata not in the default tag but in another to let apple music let me see the lyrics.

Sadly we have to workaround the lack of stamped lyrics support by putting all of it at once.

I already put it in one song as a reference for how (i think) it should look.

Image

@EliasGabr
Copy link
Author

I added the option to only download plain lyrics, you can find the exe at the bottom of the following page

https://github.com/ots-downloader/onthespot/actions/runs/12913932554

@justin025
I just now saw this and I think it would be perfect if you also made the program embed the plain lyrics in the old lyrics tag in the metadata and put the synced lyrics in the already perfect original lyrics tag. So you would have both options in the same .mp3

@justin025
Copy link
Owner

According to the ID3 spec USLT is for unsynced lyrics and SYLT is for synced. Currently the app embeds both into USLT because I don't really know how to properly format SYLT lyrics, mutagen has thrown up errors at all my attempts, ffmpeg doesn't seem to support SYLT natively, and I have yet to find a music player that even supports SYLT.

I'm going to reopen this issue so that hopefully someone with more knowledge than me on how SYLT formatting can open a pull request.

@justin025 justin025 reopened this Jan 22, 2025
@justin025 justin025 changed the title Give option for plain lyrics downloading without timestamps for local Apple Music Embed Synced Lyrics Into SYLT Tag Jan 22, 2025
@EliasGabr
Copy link
Author

Hello, my recommendation is (when you don't know how to implement SYLT) to at least embed USLT correctly, so the more used lyrics tag standard is properly supported. Instead of now, where both are half backed implementations.

Here is a screenshot of how my USLT tag looks like:

Image

@justin025
Copy link
Owner

justin025 commented Jan 25, 2025 via email

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

No branches or pull requests

2 participants