removes unnecessary height

[brunoamaral]

Link to related issue (if applicable)

#1723

Summary of proposed changes

The height setting doesn't work well with Bootstrap, removing it fixes the problem.

Checklist

• Use develop as the base branch
• Exclude the gulp build (/dist changes) from the PR
• Test on supported browsers

[skerbis]

same for UiKit

[sampotts]

Will merge this one manually. Thanks @brunoamaral !

[brunoamaral]

My pleasure, @sampotts ! 🙇

[shravan2x]

@brunoamaral @skerbis I'm the person who initially added the height: 100% in the linked PR above. Without it, the video overflows the Plyr component.

When using with Bootstrap or UIKit, could you try wrapping the Plyr container in a div?

[brunoamaral]

Hi @shravan2x , here's the code I am using right now, with the updated version of plyr:

<div class="plyr__video-embed player">
<iframe src="https://www.youtube.com/embed/rHKwIYsPXLg?origin=https://plyr.io&iv_load_policy=3&modestbranding=1&playsinline=1&showinfo=0&rel=0&enablejsapi=1" 
allowfullscreen allowtransparency=true allow=autoplay loading=lazy>
</iframe>
</div>

The result is in this link: https://brunoamaral.eu/post/algorithms-artificial-intelligence-and-us/

It seems to work well, is your suggestion to add another div around that?