greneholt / youtube5

Safari extension that converts YouTube videos on any site into HTML5 video tags
http://www.verticalforest.com
Other
111 stars 17 forks source link

Allow player to fill all available space #55

Closed joshkulesza closed 10 years ago

joshkulesza commented 10 years ago

Having the player shape itself to remove letter-boxing causes the player to appear as though it doesn't belong. Having the player fill all available space looks more intentional and less as though something was broken

joshkulesza commented 10 years ago

If you choose to merge please ignore 8d56c06 and c9d8ed3 I forgot to exclude those commits in the pull request

greneholt commented 10 years ago

Thanks for the pull request, but having the player fit the true size of the video was an intentional design decision that I still agree with. Sorry, but I don't intend to merge this.

joshkulesza commented 10 years ago

That's fine, thanks for having a look at it