[FIX] #156 Browser complains about mixed content

scaron / prettyphoto

prettyPhoto is a jQuery based lightbox clone. Not only does it support images, it also add support for videos, flash, YouTube, iFrames. It’s a full blown media lightbox. The setup is easy and quick, plus the script is compatible in every major browser.

http://www.no-margin-for-errors.com/projects/prettyPhoto-jquery-lightbox-clone/

553 stars 280 forks source link

[FIX] #156 Browser complains about mixed content #159

Open darylteo opened 9 years ago

darylteo commented 9 years ago

Original Issue

If a site is loaded using https:// youtube videos (and probably other embeds) will not load because the script hard codes http in the video URLs

The following fix is as follows:

only applies to video urls (any impact on other embeds unknown)
checks for the current document protocol (http: or https:)
if protocol is file:, then it defaults to http: as it is existing behaviour

See this fix in action here and by clicking the vimeo videos on the right sidebar

darylteo commented 9 years ago

This PR now supports the allowfullscreen property.