-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Google Cast #412
Google Cast #412
Conversation
Working on stuff
Plyr cast button now responds properly to current session state. The button now represents whether the parent plyr is the one that is currently assigned to ongoing cast session. On page load, if there is no cast session, clicking the button does two things: - Show cast drop down - Bind current plyr instance on successful session negotiation However, on page load, if there already exists a cast session, the button is still greyed out to signal that the plyr is not attached. Upon click, the plyr is bound to the cast session and only the icon is updated without going through cast connection.
The demo/cast-server.js requires 3 additional npm packages that have not been added to package.json: express, morgan, compression
Let me know if you want me to squash these commits. There is at least one unused file that made it into git history. |
any update on this ? |
There's supposedly a big update in the works. I could make progress on this, but it would just mean massive merge conflicts once that update hits. Hence, I've sort of paused work on this till the big update is up. As it is, this PR changes a lot of code since it tries to re-use a lot of the helper functions that Plyr uses. |
👍 |
As @gurupras pointed out - I'm in the middle of a large update and want to get that out ideally before any more major work is done |
Closing this and opening a new PR #706 |
Adds google cast capabilities to plyr.
Link to related issue (if applicable)
#112
Sumary of proposed changes
PlyrUtils
,PlyrCast
_injectScript
,_on
, etc) toPlyrUtils
(these are used inPlyrCast
)Task list
plyr.less
changes make sensescss
sources