Comment plugin in IE is not adjusting height

here is a patch to change the namespace.

Well shoot, changing the namespace did not fix my issue after all. For a while after I changed it my problem went away, but of course as soon as I created the patch and uploaded it, the issue returned. Might be a good idea to change the namespace back to its original state until its absolutely necessary that it needs to be changed.

Any idea why the comment plugin in IE doesn't load the height properly in IE? Here's a couple examples:
http://alms.com/alms-tv/episode/scott-sharp-fire
http://www.rightthisminute.com/video/losing-your-feet-video-games-truth-...

Each of the above URLs have multiple comments. They work fine in firefox for example. In IE, only one comment loads, or gets cropped off it the comment is too long. Reloading the same page will sometimes cause the plugin to work properly.

There is something in the Facebook API module that is causing this issue but I'm not sure what it is yet...

For testing purposes I commented out the fb_page_alter() function so that it wouldn't print the FB javascript. I then added the code that comes directly from FB's comment plugin page in a similar fashion.

function myModule_page_alter(&$page) {
  $output = \<\<\<END
    \<div id="fb-root" class="></div>
    <script>(function(d, s, id) {
      var js, fjs = d.getElementsByTagName(s)[0];
      if (d.getElementById(id)) return;
      js = d.createElement(s); js.id = id;
      js.src = "//connect.facebook.net/en_US/all.js#xfbml=1";
      fjs.parentNode.insertBefore(js, fjs);
    }(document, 'script', 'facebook-jssdk'));\</script\>
END;
  
  $page['page_bottom']['myModule'] = array(
    '#type' => 'markup',
    '#markup' => $output,
  );
}

Still didn't solve the issue, the same broken behavior was happening in IE. I then started disabling the various modules I was using in the Drupal for Facebook fieldset. Once I disabled the Facebook API module the comments started working properly. At this point, none of the FB modules are enabled.

I suspect that there may be an issue in the fb_ajax_event() function, but thats a guess at this point.

@Dave Cohen Any ideas on where to look or what might be causing the issue?

Its not a module conflict issue. The only FB module I have enabled at the moment is Facebook API. I'm also not planning on adding the scripts myself, its merely for testing/debugging purposes.

I've narrowed it down to the fb.js file that is causing the IE issue. I have commented out everything in fb.js and fb_page_alter() is still not being used and is replaced by my custom function from #6.

I'll keep digging...

I'm running out of time for today but it looks like the issue is in Drupal.behaviors.fb method. More specifically, for some reason IE will run the following chunk of code when it probably shouldn't.

if (typeof(FB) != 'undefined') {
  // Render any XFBML markup that may have been added by AJAX.
  $(context).each(function() {
    var elem = $(this).get(0);
    FB.XFBML.parse(elem);
  });
}

Commenting out the FB.XFBML.Parse(elem); line will fix the issue that IE is having upon page load. I'm still trying to figure out why IE is having issues with this.

yeah the problem happens there too. I've found a simple solution and I'm rolling a patch now.

Here is a patch that fixes the issue in IE 7, 8 and 9. In fb.js around line 305 you will find the code that was pasted in comment #9 above.

if (typeof(FB) != 'undefined') {

That line is testing if the FB object has been defined. For some reason in IE that object is defined but is essentially empty.

The solution I came up with is to check if the FB._apiKey is null.

if (typeof(FB) != 'undefined' && FB._apiKey != null) {

I'm not sure why the FB object is different between the two browsers but I do know that only the IE browsers are executing that bit during initial page load. If I reload the page, then the FB object is properly populated and comments work as expected.

IE 9 comes with the Developer Toolbar (press f12 to activate). Use this article for instructions on how to set breakpoints and watch the FB object. http://msdn.microsoft.com/en-us/library/ie/dd565625%28v=vs.85%29.aspx#_vars.

I also added in console.log statements into the fb.js code. Since IE comes with the developer toolbar, it no longer blows up when console.log is used.

That bit of code from comment #9 is used to process XFBML markup after an ajax request. Drupal.settings.fb.fb_init_settings.xfblm = false for my set up. I'm only using HTML5 tags on my site. Should there be an IF statement wrapping that code that prevents it from being executed if xfblm is false? Maybe that's a good enough fix.