Use an existing player if present when video changes. #61
Conversation
| if (obj == null) return true; | ||
| if (Object.keys) return Object.keys(obj).length == 0; | ||
| var keys = []; | ||
| for (var key in obj) if (_.has(obj, key)) keys.push(key); |
There was a problem hiding this comment.
can we do obj.hasOwnProperty(key) instead of the underscore dependency?
|
i haven't pulled this down and tested things out yet, but this looks like the right approach! thanks a lot for the help 🌟 |
|
|
||
| function loadPlayer () { | ||
| if (scope.videoId || scope.playerVars.list) { | ||
| // If present, use an existing player | ||
| if(scope.player && !isEmpty(scope.player) && scope.player.getIframe() && scope.player.cueVideoById === 'function') { |
There was a problem hiding this comment.
what does this getIframe() check do?
There was a problem hiding this comment.
Thanks for all of your hard work here @brandly @alexthewilde. scope.player.cueVideoById === 'function' doesn't work for me. It causes the check for the preexisting player to fail when a valid player is instantiated. I think you might mean (typeof scope.player.cueVideoById === 'function') here?
There was a problem hiding this comment.
@brandly I believe this method returns the DOM node for the embedded iframe, maybe the intention is to make sure that it exists/is loaded?
There was a problem hiding this comment.
The getiframe function doesn't exists. I guess alex forget this one to include
|
hey, @alexthewilde! take a look at my comments when you get a chance |
See original issue #46
Fix adapted from #48