[documentation] [bug] js addLoadEvent not working.
bryan kennedy
drupal-docs at drupal.org
Tue Feb 14 20:51:25 UTC 2006
Issue status update for
http://drupal.org/node/27884
Post a follow up:
http://drupal.org/project/comments/add/27884
Project: Documentation
Version: <none>
Component: Developer Guide
Category: bug reports
Priority: critical
Assigned to: gordon
Reported by: gordon
Updated by: bryan kennedy
Status: patch (ready to be committed)
I added this issue to the module upgrade handbook page [1]
[1] http://drupal.org/node/22218
bryan kennedy
Previous comments:
------------------------------------------------------------------------
Sun, 31 Jul 2005 05:01:37 +0000 : gordon
if there is a on_load attribute on the body tag then the events added by
the addLoadEvent() will not be run.
try adding aler_t('on_load') to the body and it will cause the failure.
darn malicious data catcher, making this post so hard to enter.
------------------------------------------------------------------------
Thu, 29 Dec 2005 17:00:35 +0000 : killes at www.drop.org
Gordon, is that still an issue?
------------------------------------------------------------------------
Fri, 30 Dec 2005 00:56:44 +0000 : gordon
Yes as far as I know, it is still an issue. This is more to do with the
browsers not running the window.onload when there is a on_load in the
page.
One option is that we obsolete the on_load and force all modules to use
the new js function.
------------------------------------------------------------------------
Wed, 18 Jan 2006 20:05:20 +0000 : gordon
Attachment: http://drupal.org/files/issues/remove_onload.patch (4.39 KB)
Removed theme('onload_attributes') as this causes the new js stuff to
die.
If you need to run some js on_load then use the new js addLoadEvent()
to start a function.
------------------------------------------------------------------------
Wed, 18 Jan 2006 20:18:04 +0000 : gordon
Attachment: http://drupal.org/files/issues/remove_onload_doc.patch (1003 bytes)
here is a patch to remove the on_load hook from the documentation
------------------------------------------------------------------------
Wed, 18 Jan 2006 21:26:46 +0000 : darius
Seems that removing hook_onload is the right thing to do at this stage.
There was a duplicate issue http://drupal.org/node/31755 which I just
closed that discussed the same problem.
Darius
------------------------------------------------------------------------
Fri, 20 Jan 2006 08:54:58 +0000 : Dries
Committed this to HEAD. Moving this to the documentation project as the
documentation patch still needs committing. I'd also add a note to the
upgrade guide.
More information about the documentation
mailing list