Reply
Thread Tools Display Modes
Unread 12-23-13, 09:41 PM   #1
Phanx
A Pyroguard Emberseer
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 4,927
iframes, stahp!

Apparently I cannot post a link to a YouTube video without the forum trying to convert it to an <iframe> embedding the linked video in the page. I hate embedded videos. I don't want to see them. I don't want to subject other people to seeing them. Please stop vBulleting from converting all YouTube URLs -- even ones I explicitly link with [url] tags and non-URL text! -- into dirty iframes!

__________________
Author/maintainer of Grid, PhanxChat, ShieldsUp, and many more.
Troubleshoot an addonTurn any code into an addonMore addon resources
Need help with your code? Post all of your actual code! Attach or paste your files.
Please don’t PM me about addon bugs or code questions. Post a comment or forum thread instead!

Last edited by Phanx : 12-23-13 at 09:43 PM.
Phanx is offline   Reply With Quote
Unread 12-27-13, 03:37 AM   #2
Phanx
A Pyroguard Emberseer
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 4,927
Also, whatever you guys are using to convert YouTube URLs into embedded videos seems to have several technical problems, in addition to the basic problem (for me) of its existence:

(1) It converts YouTube URLs that are inside a [ URL="insert link here" ] tag, resulting in an <iframe> tag in the middle of an <a> tag's "href" attribute, which breaks both the link and the (unwanted) iframe.

(2) It tries to follow short URLs to figure out if they redirect to a YouTube URL, to convert those into embedded videos... but it doesn't actually work (at least for TinyURL) and ends up making a link labeled "null" that points to "null". Example here:
http://www.wowinterface.com/forums/s...819#post288819

Can't you just add a new BBCode tag for embedding -- eg. [ embed ] or [ video ] -- instead of using this weird buggy conversion thing?
__________________
Author/maintainer of Grid, PhanxChat, ShieldsUp, and many more.
Troubleshoot an addonTurn any code into an addonMore addon resources
Need help with your code? Post all of your actual code! Attach or paste your files.
Please don’t PM me about addon bugs or code questions. Post a comment or forum thread instead!

Last edited by Phanx : 12-27-13 at 03:42 AM.
Phanx is offline   Reply With Quote
Unread 01-28-15, 04:56 PM   #3
Phanx
A Pyroguard Emberseer
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 4,927
This is still a problem over a year later, and it's still just as annoying.
__________________
Author/maintainer of Grid, PhanxChat, ShieldsUp, and many more.
Troubleshoot an addonTurn any code into an addonMore addon resources
Need help with your code? Post all of your actual code! Attach or paste your files.
Please don’t PM me about addon bugs or code questions. Post a comment or forum thread instead!
Phanx is offline   Reply With Quote
Unread 01-28-15, 08:21 PM   #4
Seerah
Fishing Trainer
 
Seerah's Avatar
WoWInterface Super Mod
Featured
Join Date: Oct 2006
Posts: 9,760
http://www.wowinterface.com/forums/s...758#post305758
__________________
"You'd be surprised how many people violate this simple principle every day of their lives and try to fit square pegs into round holes, ignoring the clear reality that Things Are As They Are." -Benjamin Hoff, The Tao of Pooh

Seerah is offline   Reply With Quote
Unread 01-28-15, 09:03 PM   #5
Phanx
A Pyroguard Emberseer
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 4,927
It shouldn't do it with any URLs. In your example post you linked, the page display is literally broken. Adding an [ embed ] tag for BBCode in vBulletin appears to be trivial (see http://www.vbulletin.org/forum/showthread.php?t=301799) so I don't see why this broken hack is still around over a year after it was reported as broken. URLs that point to images don't turn into embedded images -- there's a separage [ img ] tag for that -- so it doesn't make sense for URLs that point to videos to turn into embedded videos, especially when the implementation is broken.

If you guys are really all too busy to spend 2 minutes fixing the problem, I hereby volunteer my time, though I suspect it would actually take longer to give me the necessary permissions (and presumably remove them afterward) than it would be to just fix it yourself.

Also, replying to your post in the other thread:

Originally Posted by Seerah View Post
Off-topic: its the https links that do it.
Even an insecure HTTP link will produce an unwated effect if it's not "raw":

example

That should be a link displaying the text "example". While you could maybe argue that someone posting a raw Youtube URL in their post wants an embedded video (though I would disagree) there's just no way you can assume someone entering a Youtube URL as the attribute of a URL tag wants an embedded video -- they clearly want the contained text to be a link to the video.
__________________
Author/maintainer of Grid, PhanxChat, ShieldsUp, and many more.
Troubleshoot an addonTurn any code into an addonMore addon resources
Need help with your code? Post all of your actual code! Attach or paste your files.
Please don’t PM me about addon bugs or code questions. Post a comment or forum thread instead!

Last edited by Phanx : 01-28-15 at 09:14 PM.
Phanx is offline   Reply With Quote
Reply

Go BackWoWInterface » Site Forums » Site help, bugs, suggestions/questions » iframes, stahp!

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off