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

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. :(