-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Dailymotion Support #682
Comments
You have the best open source player yet. Adding support to @dailymotion will push you ahead away from other players, please put more efforts on this. |
+1 for Dailymotion support. |
Any updates here? We'd love Dailymotion support. |
I'm planning to add dailymotion support for my own use, should I make a PR @friday ? |
Thanks for the reply! So I have pushed a new branch to my fork of the develop branch implementing DailyMotion (https://github.com/akuma06/plyr/tree/dailymotion). I have tested it locally and I will use it for my own app. It comes down now to @sampotts wether he wants me to PR it now or wants to wait for his plugin system. :) |
Thanks @akuma06, I would love for your work to be in PR. We are starting to integrate Plyr in our application, and we would want immediate Dailymotion support. +1 from me |
I wanted to chime in here again to thank @akuma06 for the wonderful work! The Dailymotion support works perfectly, and I'm happily using it in my app through my own fork of this repo. It would be wonderful to have it merged into the official repo. |
Yes @adeeb1, It would be wonderful to have it merged into the official repo 🥺 |
@sampotts congrats on the new release. wasn't there any chance to consider this in this new release? or any upcoming ones, please? |
Happy to take a look at a PR for it... |
I don't mind doing the pr for it after I've checked if there are any changes with the new release. I didn't do it before since I thought @sampotts wanted to prioritize a plugin system before adding any new platform to the code base. |
Any updates @akuma06? |
@akuma06, Any updates on this? |
Sorry I completely forgot about this :'). I just updated the branch to last changes, I've checked and everything works, I've just sent a PR #2136 ! |
Many thanks @akuma06 ❤️ |
Sometime it's discouraging that no one or the author do not a minimal help and just fork the readme without minimal change with no help at all. |
Not sure what you're getting at but sorry, I don't have much time to devote to Plyr. I now have two kids (6 year old and 2 month old) and a full time job (SDE3 at Amazon) so I don't have a lot of spare time. I'm hoping to put some time into Vidstack which will hopefully be the logical successor to Plyr. |
Also, the current structure of Plyr means everyone gets all the logic, even for features they don't need. So, if we add every feature that anyone asks for, the player becomes bloated and folks have something new to complain about! |
I've updated the README on my fork with HTML instructions for Dailymotion here. |
I can understand that. Or giving it to the community with chosen top contributors? For my complain, it was about the one you did the Dailymotion "addon" without minimal infos on is fork. I think a plugins system like Videojs would be fun but i know, more work to come :) I checked Vime sometime days/month ago and it have good potential but there's still a lot of work to do i think to be ready et mature like Plyr. Videojs had used a lot of free contributors and took the best Videojs plugins and functions to update Videojs and their own compagny Brightcove. BTW, i'm doing a little javascript plugin to make all videos who are set to autoplay, to autoplay with sound. any gesture with a promise will make video and audio chosen (autoplay=1) on autoplay to play with sound. It will never be perfect but it's fun. i have good success with some litle test until now. Only video often on top will show a button like youtube overlay the video with a mute icon and will have a text "Unmute" in 11 languages. (sorry fo my english) have a good day |
No description provided.
The text was updated successfully, but these errors were encountered: