html.tpl.php and maintenance-page.tpl.php both have hardcoded meta tags that don't add anything to browsers except Windows Mobile see #1717090: Remove the http-equiv="cleartype" meta tag

This patch adds conditionals around them so pages will validate

CommentFileSizeAuthor
cleartype-http-equiv-fix.patch994 bytesacbramley
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

echoz’s picture

Version: 7.x-5.1 » 7.x-5.x-dev
Status: Active » Postponed

Let's follow what core does #1717090: Remove the http-equiv="cleartype" meta tag even though that's been stalled for a month. Patch will have to be rerolled against dev when the time comes. Thanks, Adam!

JohnAlbin’s picture

Version: 7.x-5.x-dev » 7.x-6.x-dev

We've decided to remove that metatag for Drupal 8 because we won't be supporting those old mobile Windows phones.

But I think we should support them on Drupal 7. Opinions?

JohnAlbin’s picture

Version: 7.x-6.x-dev » 7.x-5.x-dev

Didn't mean to move it to the new branch.

C-Logemann’s picture

Issue summary: View changes
Status: Postponed » Active

We have still validation errors on this meta tag in W3C Validator. So even if it's now removed from D8, there should be a soltion in the D7 zen theme.
I like the solution of @acbramley and the patch works fine in branch 7.x-5.x-dev. This can be committed in my opinion.

rooby’s picture

Status: Active » Needs review

There is a patch here.

JohnAlbin’s picture

I think the wrapping conditional comment is silly, but I'll add it to 7.x-5.x since I'm removing the whole metatag from 6.x.

  • JohnAlbin committed 276d637 on 7.x-5.x authored by acbramley
    Issue #1781662 by acbramley: Bad value cleartype for attribute http-...

  • JohnAlbin committed 5af761a on 7.x-6.x
    Issue #1781662: Bad value cleartype for attribute http-equiv on element...
JohnAlbin’s picture

Version: 7.x-5.x-dev » 7.x-6.x-dev
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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