I think fb_connect.js (line 11) contains bug. When document is loaded $(this).html() returns empty and next code will not be called. $(this).children().length works more correctly for me.

CommentFileSizeAuthor
#3 patch.diff840 bytesvectoroc

Comments

vectoroc’s picture

vectoroc commented

btw, I don't understand what is this condition for ? If I load via ajax some like (<fb:profile-pic uid=xxx linked=false></fb:profile-pic>) then code under condition will not be called (even $(this).html() or $(this).children().length)

Dave Cohen’s picture

Dave Cohen commented

You may be right that $(this).children().length is better. I'm no javascript expert and I need suggestions like this to get it right.

What browser are you using?

Try adding the following to FB_Connect.init() at the bottom of the file:

            // Respect fb_connect classes on new content.                       
            $('.fb_connect_show').show();
            $('.fb_connect_hide').hide();

Also I didn't understand your comment. you need to use <code> when posting here.

vectoroc’s picture

vectoroc commented
StatusFileSize
new patch.diff840 bytes

I use Safary, FF. It does not matter.
I believe that this patch is most correct way.

Dave Cohen’s picture

Dave Cohen commented
Title: Profile page does not show user picture. » XFBML not rendered.
Status: Active » Fixed

Thanks for this. Just checked it in along with some event binding improvement.

Status: Fixed » Closed (fixed)

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