fbConnect aware themes

I've checked in some features specific to Facebook Connect. In order for any of it to work, the theme must be "facebook-aware". As intimidating as this sounds, it's no big deal.

As an example, I've modified Drupal's default theme, garland, to support the XFBML login button. All it took was minor modifications to the page.tpl.php file. I'm attaching a diff, for your convenience.

AttachmentSize
garland_fb.txt938 bytes

Hello, I fixed this two way

Hello,
I fixed this two way in page.tpl.php

   1.xmlns:og="http://opengraphprotocol.org/schema/" xmlns:fb="http://www.facebook.com/2008/fbml"
   2.<!-- http://wiki.developers.facebook.com/index.php/JavaScript_Client_Library -->
<div id="FB_HiddenIFrameContainer" style="display:none; position:absolute; left:-100px; top:-100px; width:0px; height: 0px;"></div>

But still not working in IE.
How can I do for working on IE?
(Sorry,my English is very poor...>"<.Hope my problem you can understand.)

im getting an error when

im getting an error when trying to patch so I did a manual patch and now seeing this error in drupal 6.17

recoverable fatal error: Object of class stdClass could not be converted to string in /var/www/beta/themes/garland/page.tpl.php on line 5.

line 5 had this added to it xmlns:fb="http://www.facebook.com/2008/fbml"> as suggested in the patch..

Did you ever fix this?

OMG I am getting the same error now, facebook icons now appearing but same error in IE. Did you ever get this working on your site?

samples

sample test FB modules

is it fixed?

i did a manual patch and i am not seeing fb connect button in any of my browsers (chrome, ie, firefox)

Thanks

The changes you recommended worked perfectly.

-Anil

add this to INSTALL.txt

Would be great to have this info in install.txt of fb module :)

Way cool

very cool, thank you

Nice!

Nice!

hook_footer?

Why not just do the bottom two items via hook_footer() in the module?

Honestly, I'm not even sure

Honestly, I'm not even sure those bottom lines are necessary. See the URL in the comment for details.

If they were in hook_footer, they'd be competing for the bottom spot with other modules, depending on the weights of modules implementing hook_footer. As the patch is, I'm guaranteed it will be at the very end.

Same Error

I did everything you said, but I still having the same warning message:

* warning: Invalid argument supplied for foreach() in .../includes/theme.inc on line 463.
* warning: Invalid argument supplied for foreach() in .../includes/theme.inc on line 468.

Bottom two lines aren't

Bottom two lines aren't necessary, one is a comment, the other is a div tag. I'm having a similar problem, though, and I'm looking for the solution, but it's taking forever to find. :(