Closed (fixed)
Project:
Drupal for Facebook
Version:
7.x-3.x-dev
Component:
Facebook Connect
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
19 Feb 2012 at 15:42 UTC
Updated:
2 Apr 2012 at 16:30 UTC
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
Comment #1
letapjar commentedoops! 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
Comment #2
bschilt commentedChanging the namespace fixed the issue I was having in the 7.x branch. #1480868: Comment plugin in IE is not adjusting height
Comment #3
Dave Cohen commentedTheir 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.
Comment #4
Dave Cohen commentedChanged the readme.txt in D6 and patched D7.
Thanks.
Comment #5
imoreno commentedWell, 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
Comment #6
joeyda3rd commentedSince other modules use ogp.me declaration, both declarations are now included with the same attribute. This creates an html validation error.
Comment #7
Dave Cohen commentedI've changed it back, does latest .dev build fix the problem?