Drigg youtube embeded teaser code not XHTML compliant

daveLA - November 1, 2008 - 04:26
Project:Drigg
Version:5.x-1.36
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

The Drigg embeded youtube thumbnails are not compliant to XHTML standards. Here are my proposed changes to make them compliant.

I've made these changes on my Drigg site: http://www.bananacut.com
and as you can see, they are compliant! http://validator.w3.org/check?uri=http%3A%2F%2Fwww.bananacut.com%2F

This is my first time creating a patch so let me know if I did something incorrectly.

-Dave

AttachmentSize
youtube_embed_valid_xhtml.patch6.37 KB

#1

mercmobily - November 2, 2008 - 12:35

Hi,

Waiting for the release of Drigg for D6, coming up in the next 3/4 days. Will apply then!

(Please bump this if I forget...)

Merc.

#2

mercmobily - December 9, 2008 - 11:34

Hi,

Is anybody able to check if this patch applies cleanly to D5 and D6? Please?

Thanks!

Merc.

#3

philbar - June 16, 2009 - 03:56
Status:active» fixed

Commit #225772 by philbar at 03:46

NOTE: The edit to the Drigg UI module was unintentional and was reverted by Commit #225774 at 03:53

#4

mercmobily - June 18, 2009 - 07:32
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.