search

esotalk / esoTalk

Fat-free forum software.
GNU General Public License v2.0
1.47k stars 239 forks source link

update YouTube embedding #432

Closed mroi closed 9 years ago

mroi commented 9 years ago
  1. Use a https URL within the embedding iframe. Otherwise, modern browsers issue mixed content warnings, when esoTalk itself runs over TLS.
  2. Change the regex to extract any additional options and pass them along to the iframe URL. This allows to embed YouTube playlist URLs, which contain a list= argument in addition to the video ID.
mroi commented 9 years ago

I have no idea why this PR got closed. It is certainly still valid.