What in the world...?

[JermCool]

Okay, fine. We can discuss our future while being spanked. Will that make you happy?

[Dreamer]

Maybe.

It just felt like the spankings were disingenuous is all...

[Martin Blank]

I wonder if 3.2 will allow youtube embedding in browsers outside of IE?

Like this?



The BBCode needed an update. The BBCode for what's above is:

Code: Select all

[youtube]https://www.youtube.com/watch?v=P99qJGrPNLs[/youtube]

I think everyone should be able to use it, based on the permissions I see. If not, let me know and I'll have another look at it.

[Cyberliger777]

You marvelous bastard. Also, I see why you raved about Cyberpunk 2077 now, that looks cool.

[JermCool]

And this is why MB is glorious to behold!

Holy crap, that looks bad-ass.

[Deacon]

So now we should use the full URL, not just the video ID?

Edited for testing:

[youtube]P99qJGrPNLs[/youtube]

Edited for results: well, yeah, apparently now is the full URL. Good to know! I wonder what happens to previous posts.

[Martin Blank]

They're broken, and I could fix them, but I'm not sure there's a major reason to do so. Or I can go back and fix it (maybe), but use a different BBCode for full URLs. That would just add confusion.

I suppose I could technically create a new BBCode for the old ones and do a SQL query to fix them, but that could get ugly.

[NorthernComfort]

I suppose I could technically create a new BBCode for the old ones and do a SQL query to fix them, but that could get ugly.

[Martin Blank]

Actually, I can probably hit up Brian to put together the code for it. He's become something of a name in the JavaScript coding world.

[NorthernComfort]

[youtube]dQw4w9WgXcQ[/youtube]

[NorthernComfort]

Code: Select all

document.addEventListener('DOMContentLoaded', function() {
[].slice.call(document.getElementsByClassName('postcontent')).forEach(
  function(content) {
    var html = content.innerHTML;
    if (html.match(/\[youtube\]/)) {
      content.innerHTML = html.replace(/\[youtube\]\s*(\w+)\s*\[\/youtube\]/g,
      '<iframe width="560" height="315" src="https://www.youtube.com/embed/$1" frameborder="0" allowfullscreen></iframe>');
    }
  });
});

Paste that into some global JS file and it should Just Work™. Copy paste the inner function (ignore first and last line) into your dev console to try it out.

[Martin Blank]

It's not quite that simple. Here's the BBCode usage:

Code: Select all

[youtube]{IDENTIFIER}://{SIMPLETEXT1}youtube.com/watch?v={SIMPLETEXT2}[/youtube]

And here's the HTML replacement:

Code: Select all

<object width="425" height="350"><param name="movie" value="{IDENTIFIER}://{SIMPLETEXT1}youtube.com/v/{SIMPLETEXT2}"></param><param name="wmode" value="transparent"></param><embed src="{IDENTIFIER}://{SIMPLETEXT1}youtube.com/v/{SIMPLETEXT2}" type="application/x-shockwave-flash" wmode="transparent" width="425" height="350"></embed></object>

That is, of course, the Flash version. I'd prefer to have the HTML5 version, which I think is what you provided, NC. If you can tweak it to work with that, I'll get it in place.