When i try to valid the html code using the w3 validator i have this error:

Line 249, Column 34: required attribute "type" not specified

The attribute given above is required for an element that you've used, but you have omitted it. For instance, in most HTML and XHTML document types the "type" attribute is required on the "script" element and the "alt" attribute is required for the "img" element. Typical values for type are type="text/css" for and type="text/javascript" for . http://validator.w3.org/check?verbose=1&uri=http%3A%2F%2Fviajesmonllar.com%2F
CommentFileSizeAuthor
#11 888756-9-facebook-validation.patch20.77 KBbronzehedwick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Carlitus’s picture

Carlitus CreditAttribution: Carlitus commented

umm...don't check the link..i've disabled the module meanwhile

ferdi’s picture

ferdi CreditAttribution: ferdi commented
Status: Active » Postponed (maintainer needs more info)

What's the line that doesnt validate ?

minus’s picture

minus CreditAttribution: minus commented 
xmlns:fb
xmlns:og
<fb:like-box>
<fb:share-button>
<fb:comments>
A tag is not recognized. It is not a know tag, in the W3c specification or a know Netscape or Internet Explorer proprietary extension.

-Tidy Validator for Firefox

Carlitus’s picture

Carlitus CreditAttribution: Carlitus commented

it's a script tag without a type="text/javascript"

<div id="fb-root"></div><script>
Rob C’s picture

Rob C CreditAttribution: Rob C commented
Version: 6.x-1.0-beta4 » 7.x-2.x-dev
Status: Postponed (maintainer needs more info) » Active

Can we try something like this?

http://www.liormark.com/develop/development-articles/facebook-fbml-w3c-v...

Cause tested it, kinda works (to get rid of the validation error) But i ran into some issues on placing the script tag and working together with the core libraries module. So no patch yet.

More info: http://stackoverflow.com/questions/2761622/new-facebook-like-button-html...

> Bump to 7.x? (also suffers from this issue)

jscm’s picture

jscm CreditAttribution: jscm commented

Subscribe..
I have the same problem and I don't know how can I solve it.
To much errors from fb code.

Do you have a real solution?

jscm’s picture

jscm CreditAttribution: jscm commented
Title: HTML not valid » W3C Validator and Facebook Plugin code: too much errors
Version: 7.x-2.x-dev » 6.x-1.0-beta9
Priority: Normal » Major
jscm’s picture

jscm CreditAttribution: jscm commented

I read that using fb like button inside javascript the w3c validator check it with success.

ferdi’s picture

ferdi CreditAttribution: ferdi commented
Priority: Major » Normal

There is not much we can do about it http://stackoverflow.com/questions/2761622/new-facebook-like-button-html...

Rob C’s picture

Rob C CreditAttribution: Rob C commented

The Service Links module does have a 'facebook like button' that vaidates. Maybe that's worth the look?

bronzehedwick’s picture

bronzehedwick
he/him
Location New York
CreditAttribution: bronzehedwick commented
FileSize
888756-9-facebook-validation.patch20.77 KB

But doesn't that link show that wrapping the meta tags in a comment works?

I tested on the site I'm working on and it seems to work fine.

I've attached a patch for review.

bronzehedwick’s picture

bronzehedwick
he/him
Location New York
CreditAttribution: bronzehedwick commented
Status: Active » Needs review
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Issue summary: View changes

Closed a duplicate: #1379208: Update to Drupal Coding Standards

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Version: 6.x-1.0-beta9 » 7.x-2.x-dev
Status: Needs review » Needs work

A few things. This needs a little more work, specifically PHP arrays should have a trailing comma on the last item; also, each comment needs to be wrapped at 80 chars. Furthermore, the changes should start with the 7.x-2.x branch and work downwards.