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

Lyrics search support #336

Closed
hbarylskyi opened this issue Jul 29, 2016 · 5 comments
Closed

Lyrics search support #336

hbarylskyi opened this issue Jul 29, 2016 · 5 comments
Labels
feature request Issue is related to a feature in the app waiting for author If the author doesn't respond, the issue will be auto-closed. Otherwise the label will be removed.

Comments

@hbarylskyi
Copy link

Hi. I'd like to add a feature which would allow a user to see lyrics of songs (if user is watching a music video).

That's how I see it working:
While watching a video, user would click a button. My code would parse the title of the video, look for lyrics in online databases and, in case something was found, a partially transparent box of text with lyrics would show up on the screen. User would be able to scroll the text and close the box.

What do you think about this feature? Any thoughts, suggestions?

As far as I understand, Exoplayer is coming to replace the old videoplayer. So should i try adding this "lyrics" button to Exoplayer?
Any suggestions on where/how should I add this button?

P.S. Maybe I would also be able to add subtitles support in similar way (issue #55)

@theScrabi
Copy link
Member

Well feel free to try it. But that thing with subtitles I would forget until we have exoplayer.

@stale

This comment has been minimized.

@stale stale bot added the stale label Dec 1, 2018
@Stypox Stypox added the feature request Issue is related to a feature in the app label Jun 22, 2019
@TobiGr TobiGr removed the stale label Jul 19, 2019
@EmanuelLoos
Copy link

EmanuelLoos commented Dec 4, 2020

QuickLyric is also GPLv3, available on F-Droid and supports a few different Lyrics Platforms which means it's code could be used in NewPipe.

However it doesn't seem to be completely GPLv3-or-later, so it might be good to check that back before using its code to keep NewPipe compatible with newer GPL licenses. (I checked one file which was GPLv3-or-later licensed)

Even though you could implement its code I think it might also be a good way to keep Lyrics a separate option:

  • QuickLyric seems to have some GPLv3-only parts which could cause problems whenever a GPLv4 comes out.
  • Extra apps for extra functionality would enable NewPipe to stay lightweight to still run on older devices (storage, performance)

I actually used QuickLyric with NewPipe before, however there are some errors (mostly at YT(-Music)'s tagging and NewPipe's extraction of it) which could be fixed:

  • "Auto generated by YouTube" on YT Music
    • YouTube adds " - Topic" to the YT channel if the artist doesn't have a YT channel, when playing in background NewPipe presents the YT channel's name as the artist's name and QuickLyric won't find Lyrics for a song by "Any Name - Topic"
    • If the artist does have a YouTube channel which's name differs from the real artists name QuickLyric also won't find lyrics
      • How to fix: This issue can easily be fixed by using the artist's name provided by YT Music (instead of the channel's name provided by YT) as the YT Music search function already does.

If you like you could also implement an API into QuickLyric and NewPipe to show them directly (and more smooth) in NewPipe.

@litetex
Copy link
Member

litetex commented Oct 20, 2021

I think we have subtitles now on most videos which should be sufficient to display the lyrics of a song (if they provided the lyrics in the subtitles).
Otherwise I think this is out of scope.
If you want to search for a lyrics use a website or app that is specialized for that.

@litetex litetex added the waiting for author If the author doesn't respond, the issue will be auto-closed. Otherwise the label will be removed. label Oct 20, 2021
@github-actions
Copy link

github-actions bot commented Nov 4, 2021

This issue has been automatically closed because there has been no response to our request for more information from the original author. With only the information that is currently in the issue, we don't have enough information to take action. Please reach out if you have or find the answers we need so that we can investigate further.

@github-actions github-actions bot closed this as completed Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Issue is related to a feature in the app waiting for author If the author doesn't respond, the issue will be auto-closed. Otherwise the label will be removed.
Projects
None yet
Development

No branches or pull requests

6 participants