1. Enable HTML5 Tools module, go to the setting, check Alter publication date markup and press Save.
  2. Open any page with comments. Subbmitted information will be like Submitted by admin on <time pubdate="pubdate" datetime="2013-01-06T18:58:07+04:00">Sun, 01/06/2013 - 18:58</time>

HTML5 Tools Adaptivetheme bug

html5_tools_apaptivetheme_bug.png8.79 KBPlazik


Jeff Burnz’s picture

Category: bug » feature
Status: Active » Closed (won't fix)

AT already uses time element and the correct date time functions, so no point in supporting this module.

Also html5 tools does not use the correct ISO8601 date time format, instead they use format c which is wrong.

hass’s picture

Category: feature » bug
Status: Closed (won't fix) » Active

Why is c wrong? Per http://www.php.net/manual/en/function.date.php it's the iso8601 format.

Aside of this it would be good to fix these theming bug.

Jeff Burnz’s picture

Category: Bug report » Feature request
Issue summary: View changes
Status: Active » Closed (works as designed)

I mixed that up regarding the formats, sorry I have been pretty ill the past while and I did not check just went on vague memories. I did extensive research on this at the time and ended up writing a custom date format just for datetime, because the html5 requirements for datetime attribute are very specific and it had to be a custom format: 'Y-m-d\TH:i:s\Z'. Specs may have changed, but at the time its what I had to do.

Anyway, I am not going to change this now because there is no real bug and changing it now is just far too late, it was at the time this was first posted which is why I closed it. What the OP is asking is to remove the time element, which can't fly due to existing themes.

hass’s picture

Category: Feature request » Bug report
Status: Closed (works as designed) » Active

There is a bug as HTML code is checkplained. This is clearly a bug. Please see the screenshot. HTML code must never been shown on a page.

Jeff Burnz’s picture

Category: Bug report » Feature request
Status: Active » Closed (works as designed)

Since we removed the permalink on the comment submitted by date (aka wordpress style of yore) there is no sanity check, what will happen how if you turn on the html5 tools option to use time element it will get nested inside the theme provided time element, the fix for that is turn off the feature in html5 tools, the theme already provides this element on date time in comments, articles and aggregator items etc.