I have a module that does an ajaxForm call to get a new version of the block's page, and then does a .html() call to replace the current blocks table with the new one. Then, of course, I do a Drupal.attachBehaviors on the new table. That is all fine and the javascript table headers show up as expected on the new table. But, as soon as I scroll, I get the following error in Firebug:
parent has no properties
while ( parent.tagName && !/^body|html$/i.test(parent.tagName) ) {
in jQuery 2958
(I got the actual jQuery line and line number by swapping in an uncompressed version of jQuery 1.2.1. It is in the offset function).
I commented out the contents of the tableheader.js file, and the error went away, but I have not been able to track it down more than that.
I have not noticed any actually problems or reduced functionality as a result of this error.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | nopersist-select-onresize.patch | 613 bytes | dvessel |
Comments
Comment #1
dvessel commentedThis was originally encountered here: http://drupal.org/node/179937
That was when the blocks table was implemented with disabled regions not showing up on the table. When I was fixing various other issues with tableheader.js, I reverted that by accident thinking I was using a more efficient selector.
Could you do a quick test by swapping out this line at the bottom of tableheader.js?
Comment #2
dvessel commentedHere's a patch instead.
Performance isn't a problem since it's on resize which is seldom done. Should have seen this earlier. Sorry.
Comment #3
starbow commentedCool. The patch fixed the error message, and the table headers still seem to work fine.
Thanks!
Comment #4
starbow commentedSince this is a regression, I am going to bump it up to critical.
Comment #5
dvessel commentedI don't think it's critical. But you demonstrated that it fixed the problem. Marking RTBC.
Comment #6
gábor hojtsyCommitted, thanks.
Comment #7
starbow commentedDrat, the parent has no properties error has started showing up again. I will attempt to create a good test case to share.
Comment #8
starbow commentedhttp://drupal.org/node/214329
Comment #9
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.