plyr 3.5.3 Encountered error fullscreen in mobile

Created on 21 Apr 2019  ·  4Comments  ·  Source: sampotts/plyr

I've just upgraded plyr 3.5.2 to plyr 3.5.3 for the project 102tube and have discovered a bug fullscreen in mobile error.

And this error causes from the Plyr JS Library when manually adding
padding-bottom: 216.533%
In div class: .plyr__video-wrapper, .plyr__video-embed

And the way I can fix that is to add to this CSS snippet
.plyr__video-wrapper, .plyr__video-embed { padding-bottom: 56.25% !important; }

Video bug : https://www.youtube.com/watch?v=zS584wonJDk

If anybody is being like me or referencing Nhé, and am looking forward to plyr sharing fix this error because Plyr is an amazing library!

Bug Replicated

All 4 comments

I've got a fix lined up for this. I can probably deploy later today or tomorrow.

Your code snippet fix would work fine for 16:9 but it disregards the ratio config option. I've found a way to make the ratio "auto" for YouTube as well like HTML5 and Vimeo already are.

Fixed in v3.5.4 👍

Fixed in v3.5.4 👍

Awesome! Please update readme file to use version 3.5.4.

Awesome! Please update readme file to use version 3.5.4.

Good point. That's supposed to be automated. Whoops 🤦‍♂️

Was this page helpful?
0 / 5 - 0 ratings

Related issues

onigetoc picture onigetoc  ·  3Comments

elliottcoe picture elliottcoe  ·  4Comments

jwjcmw picture jwjcmw  ·  4Comments

ahmadshc picture ahmadshc  ·  3Comments

nam-co picture nam-co  ·  4Comments