sampotts / plyr

A simple HTML5, YouTube and Vimeo player
https://plyr.io
MIT License
26.38k stars 2.92k forks source link

Dailymotion Support #682

Open swiretvweb opened 6 years ago

abazeed commented 5 years ago

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.

kevinfaveri commented 4 years ago

+1 for Dailymotion support.

adeeb1 commented 4 years ago

Any updates here? We'd love Dailymotion support.

akuma06 commented 4 years ago

I'm planning to add dailymotion support for my own use, should I make a PR @friday ?

friday commented 4 years ago

@akuma06 It's up to @sampotts. There were architectural reasons we wanted that to be moved to plug in support. Maintaining providers is hugely problematic. I don't think that has changed, but I'm no longer contributing to plyr's development.

akuma06 commented 4 years ago

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. :)

adeeb1 commented 4 years ago

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

abazeed commented 4 years ago

Hello @sampotts, are you planning to pull @akuma06 's changes? We are patiently waiting to have this feature live and hope to push this a step forward at the soonest. Many thanks.

adeeb1 commented 4 years ago

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.

abazeed commented 3 years ago

Yes @adeeb1, It would be wonderful to have it merged into the official repo 🥺

abazeed commented 3 years ago

@sampotts congrats on the new release. wasn't there any chance to consider this in this new release? or any upcoming ones, please?

sampotts commented 3 years ago

Happy to take a look at a PR for it...

abazeed commented 3 years ago

Happy to take a look at a PR for it...

Thank you @sampotts! @akuma06, could you please PR your changes?

akuma06 commented 3 years ago

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. Will submit a pr this week :).

abazeed commented 3 years ago

Any updates @akuma06? I know you may be busy with other work, and I hope you find the time for this.

abazeed commented 3 years ago

@akuma06, Any updates on this?

akuma06 commented 3 years ago

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 !

abazeed commented 3 years ago

Many thanks @akuma06 ❤️

ghost commented 2 years ago

Hello @akuma06, @abazeed and others, can you please help me on how to use it on the HTML side?

adeeb1 commented 2 years ago

Hello @akuma06, @abazeed and others, can you please help me on how to use it on the HTML side?

You would use it just like you use the standard library. I'd need more details to provide specific help.

onigetoc commented 1 year ago

Hello @akuma06, @abazeed and others, can you please help me on how to use it on the HTML side?

You would use it just like you use the standard library. I'd need more details to provide specific help.

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.

sampotts commented 1 year ago

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.

sampotts commented 1 year ago

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!

adeeb1 commented 1 year ago

Hello @akuma06, @abazeed and others, can you please help me on how to use it on the HTML side?

You would use it just like you use the standard library. I'd need more details to provide specific help.

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.

I've updated the README on my fork with HTML instructions for Dailymotion here.

onigetoc commented 1 year ago

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.

I can understand that. But with a popular repo like this, you may eventually have to add some good contributors who have more time to spend on it? Like, the guy who made the Dailymotion addon for example, and i'm sure you spotted some good one until now".

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. Let's go, may be 30 more minutes or less of work in a lifetime.

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