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

Allow to search videos for the current song #316

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rivaldi8
Copy link
Contributor

It opens the YouTube application with a search of the song that's currently playing.

My original idea was to show the video of the song embedded in the application. That would be done showing the first video in the search results of the song. However, more often than not, the first one is not the official video for the song. Usually being just a static image instead of a real video. This happened even with well-known songs.

Also, I thought that showing search results instead of directly a video allows users to better choose what they really want. For example, sometimes I like to watch live performances of a song instead of its video clip.

It opens the YouTube application with a search of the song that's
currently playing.
@shailandraRajput
Copy link

Hello This Feature is nice...
i have a one suggestion If you can add this feature
A option for Search Lyrics in now playing style like search video.
if user click on search lyrics, a browser will be open and search for song lyrics on google...
Thanks

@naman14
Copy link
Owner

naman14 commented Nov 4, 2017

I will be merging this. Thanks for adding this.
@shailandraRajput Will be merging #250 for adding lyrics support

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.

None yet

3 participants