Here’s a simple, more flexible way to reference the Vimeo player to work with sites/pages running under https:// without console errors.
More generally, using // (instead of hard coded http://) should be considered in most places within code where a schema appears.
Please consider this patch below (changing 2 characters). If you need a Git patch then just let me know.
Edit config-templatebuilder/avia-shortcodes/slideshow.php
Go to function avia_slideshow_video_helper()
Change
http://player.vimeo.com/video/'.$video_url
to
//player.vimeo.com/video/'.$video_url
Note that this gets rid of the “mixed content” warnings. However, Vimeo’s CDN’s may still cause a mixed content icon in the browser URL bar, and in the console due to http://vimeo.com/forums/topic:116216 However, Vimeo is supposedly fully supporting https:// video “soon”, so the fix above is in preparation for that inevitability.
Hey Jonathan!
I’ll tag the topic for Kriesi to take a look at. I’m a bit on the fence as it *should* work but we’ve had a ridiculous amount of issues with youtube and Vimeo video support when the general batch of users start playing with things. It makes me a bit nervous to roll out changes these days since we’ve finally got the video stuff stable :)
Thank you!
Regards,
Devin