It appears that the xml namespace declaration for "fb" has changed. The old namespace - to support fbml

was: xmlns:fb="http://www.facebook.com/2008/fbml

According to this page on developers.facebook.com the new namespace declaration to support xfbml for social plugins should be:

(click on the "get code" button to see the xfbml code and the namespace they suggest)

I believe that the docs for dff should be updated to reflect this change, but I am unclear whether other parts of the module still use the deprecated fmbl tags so I have not made documentation changes myself.

Comments

letapjar’s picture

oops! Looks like the namespace declaration was scrubbed - it should read:

xmlns:fb="http://ogp.me/ns/fb#"

this should be inserted into the < html > tag of the page template

bschilt’s picture

Changing the namespace fixed the issue I was having in the 7.x branch. #1480868: Comment plugin in IE is not adjusting height

Dave Cohen’s picture

Their doc still includes plenty of examples of the old attribute.

Still, I will update the README.txt. Not sure what else I can do in the module.

Dave Cohen’s picture

Status: Active » Fixed

Changed the readme.txt in D6 and patched D7.

Thanks.

imoreno’s picture

Status: Fixed » Needs review

Well, since it is facebook - everything is possible...

with the last 3/14/12 dev - if you will change this declaration FB connect will not work on IE with the new declaration.
xmlns:fb="http://ogp.me/ns/fb#"
If you stay with the old one,
xmlns:fb="http://www.facebook.com/2008/fbml"
it works very nice....
I think we should wait with this. I'm keeping the old declaration as it works for me.

tested on IE8, IE9
Itzhak

joeyda3rd’s picture

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

Since other modules use ogp.me declaration, both declarations are now included with the same attribute. This creates an html validation error.

Line 13, Column 57: duplicate specification of attribute "xmlns:fb"
  xmlns:xsd="http://www.w3.org/2001/XMLSchema#"xmlns:fb="http://ogp.me/ns/fb#">
Dave Cohen’s picture

Status: Needs review » Fixed

I've changed it back, does latest .dev build fix the problem?

Status: Fixed » Closed (fixed)

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