-
Notifications
You must be signed in to change notification settings - Fork 47
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
Browser: Youtube and other sources integration #86
Comments
I don't see how this could work if you watch a youtube video in the browser. Like you say, no internal browser. How we could make this work, for example, is embed the youtube video in Lyricfier I suppose. |
@BenjaVR I don't think embedding media is what we need here. After all, the app is meant for displaying lyrics. Bundling it with a media player will only make it more heavy and slow. I have had a look at BeardedSpice's core logic and it seems they are getting tab info by running an internal AppleScript hack to get browser's foremost window information. I checked further and it's possible actually, like this. Now the question is, is it possible to embed AppleScript in Electron. @emilioastarita any idea ? |
Remember that we need to support 3 platforms. The correct path for me is to create a chrome/firefox extension to implemente native messaging: https://developer.mozilla.org/en-US/Add-ons/WebExtensions/Native_messaging |
I am not a fan of browser extension, but I agree that it may be the most feasible solution for cross-platform. So, shall we get started on this ? To publish the extension we may need a separate developer account for the app also. |
Hi, I have had this idea for a while, how hard do you think it will be to fetch current song being played on youtube and display the lyrics ? Will make this app a lot more useful, since there are many youtube songs that spotify don't have.
I did a quick search and a straightforward way seems hard to find, since there is no internal server like the Spotify case. But I have seen it workable under BeardedSpice which I am using, so there must be some way. Feel free to share what you know - I can also help with a PR if you're busy.
The text was updated successfully, but these errors were encountered: