-
Notifications
You must be signed in to change notification settings - Fork 15
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
Playlist (or category.load) missing playlist container div? #77
Comments
I am also having the same issue as brandon you can see the difference between the playlists implemented on brightcovepage here |
Ran into the same issue. i wanted the vertical playlist using react-player-loader. i added a state for the side panel button
then in my jsx return i added a wrapper, the close btn and an event handler
and finally the scss for it
Ill likely add a bit of work to get some css animations, but this should work as a base |
@brandonburkett Have you got the solution? |
I did not and we ended up back-logging the feature for now. With brightcove player 7 on the horizon, I am sure the DOM / classes will be changing up some more. Hopefully brightcove updates this library. |
Hello,
When using with the following config (including playstListId and embedOptions.playlist: true). EX:
It seems to load
div.vjs-playlist
as expected, but looks like it is missing the container div fordiv.vjs-playlist
. For example, if using the vertical playlist setting for the player and the advance embed, it looks likevjs-playlist
should be the first child ofdiv.vjs-playlist-sidebar vjs-playlist-sidebar-vertical vjs-playlist-show-hide-hidden
... but this div wrappingvjs-playlist
is missing, so styling is pretty off.Unsure if I am missing a config setting, but the ending playlist markup is not matching the advance embed.
Thanks,
Brandon
The text was updated successfully, but these errors were encountered: