haiwen / seahub

The web end of seafile server.
seafile.com
Other
525 stars 369 forks source link

[Improvement] Use plyr instead of Video.js on the download page #4109

Open doliG opened 4 years ago

doliG commented 4 years ago

I propose to change the video player used in the preview page.

I got few bugs on the current one, and find it ugly and not user friendly to use. After few research, I find a player which looks lot better: Plyr.

Why ?

Pros:

Cons:

Resources

Final words

What do you think ? Good idea or useless feature ? Tell me, I'm open :)

doliG commented 4 years ago

Hello :wave: @caixiangyue @freeplant @killing @LeoSirius @lins05 @poetwang

doliG commented 4 years ago

Hello 👋 @caixiangyue @freeplant @killing @LeoSirius @lins05 @poetwang

Doing open source also means answer to users suggestions, and I see that most of issues openend here stay without answers.

Maybe I missed something, and there is a better way to contact you ?

Thanks.

freeplant commented 4 years ago

We think plyr is optimized for youtube, which is useless for us. We will keep using video.js

doliG commented 4 years ago

We think plyr is optimized for youtube

I think you misunderstood. Plyr is a video player which support youtube. But it's just a feature, as written in the first lines of Plyr docs.

It's an easy way to improve user experience and to reduce bundle. If you want I can create a PR this way you can test.

freeplant commented 4 years ago

As the JS file is only used in video view page, the size of the library is not important.