document.body

should be

document.getElementsByTagName('body')[0]

CommentFileSizeAuthor
#2 patch_82.txt1.01 KBcasey
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RobRoy’s picture

RobRoy CreditAttribution: RobRoy commented
Status: Active » Postponed (maintainer needs more info)

It would be helpful to submit a patch according to http://drupal.org/diffandpatch. Thanks.

casey’s picture

casey CreditAttribution: casey commented
Status: Postponed (maintainer needs more info) » Needs review
FileSize
patch_82.txt1.01 KB

Ok here it is

ChrisKennedy’s picture

ChrisKennedy CreditAttribution: ChrisKennedy commented

Honestly, it would probably be best to convert that entire function to use jQuery.

Steven’s picture

Steven CreditAttribution: Steven commented
Version: 6.x-dev » 5.x-dev
Status: Needs review » Needs work

We can see about jquery methods like height() for 6.0, for now we should just fix it. Let's use $('body')[0] though, it's shorter.

Steven’s picture

Steven CreditAttribution: Steven commented
Status: Needs work » Fixed

Fixed with jQuery.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)