First of all, I am very new to Drupal, and I love the work you guys are doing to bring Drupal up to HTML5 standards. I discovered a stray double quote in the html_tools.module file on line 199, as part of the theme_html5_tools_time() function. After removing this stray quote and disabling the RDF module I was actually able to get my site to validate as HTML5 using Drupal. I have attached a copy of the fixed file to this post.

CommentFileSizeAuthor
html5_tools.zip2.74 KBFirmy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Firmy’s picture

Firmy CreditAttribution: Firmy commented
Project: HTML5 Base » HTML5 Tools
Version: 7.x-2.x-dev » 7.x-1.x-dev
Component: Miscellaneous » Code

I didn't realize I accidentally posted this in the HTML5 Base thread, so I moved it to the HTML5 Tools thread as appropriate.

ericduran’s picture

ericduran CreditAttribution: ericduran commented

Hey Firmy,

Thanks for the issue. Just and FYI for feature contributions it'll be great if you provide patches. Have a look at http://drupal.org/node/1054616

Thanks.

ericduran’s picture

ericduran CreditAttribution: ericduran commented

I see the quote you mention at line 199 

  return '<time' . drupal_attributes($attributes) . '">' . $variables['date'] . '</time>';
ericduran’s picture

ericduran CreditAttribution: ericduran commented
Status: Active » Fixed

@Firmy,

Thanks for the issue. This is now fix in the repo.

Commit -- http://drupalcode.org/project/html5_tools.git/commit/bede270a6d0c5c174eb...

Automatically closed -- issue fixed for 2 weeks with no activity.