Hello,

I noticed we need to manually include a line in our page.tpl.php to amke FB Connect work in IE8.
Could the FB Connect module do this automatically / programmatically?
#669712: FB Connect Login odd behavior when third party cookies disabled

CommentFileSizeAuthor
#6 727450_rdf_README.patch763 byteswesnick
#4 fb.727450-1.diff793 bytesekes

Comments

Dave Cohen’s picture

Only if the theme supports it. I don't know of any that do, or any standard way to do this.

Would probably require a change to core themes.

giorgio79’s picture

Status: Active » Fixed

Thanks Dave, yes I was looking around as well, it seems there is no other way at the moment.

Status: Fixed » Closed (fixed)

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

ekes’s picture

Version: 6.x-3.x-dev » 7.x-3.x-dev
Status: Closed (fixed) » Needs review
StatusFileSize
new793 bytes

Adding namespaces is better supported in 7.x because of the work done round RDF(a).
Suggest using the RDF module (core) hook - patch attached.
If folks don't have it enabled, then they will have to do it manually... which requires different instructions for a 7 theme I believe.

Dave Cohen’s picture

Assigned: Unassigned » Dave Cohen
Category: feature » task

Thanks ekes, I knew this was possible in D7 but I wasn't sure how. I've updated my local copy and will check in if no problems.

wesnick’s picture

StatusFileSize
new763 bytes

I see this has been commited. As far as I can tell it works on my D7 instance with the latest HEAD. Attaching suggested patch for the README file against 7.x-3.x-dev

Dave Cohen’s picture

Status: Needs review » Reviewed & tested by the community

I'll probably just delete the entire clause from README.txt. WIll check in shortly. Thanks.

Dave Cohen’s picture

Status: Reviewed & tested by the community » Fixed

Cleaning issue queue.

Status: Fixed » Closed (fixed)

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