This project is archived and is in readonly mode.
[Prototype 1.6.0.3] IE throws js error on unload in Events.element
Reported by skaue | October 2nd, 2008 @ 09:33 AM | in 2.x
This is a bug that survived the patch from 1.6.0.2 that should solve this problem for various other events. I'm tracing this using Visual Studio and line 4503 in prototype.js says:
if (node.nodeType == Node.TEXT_NODE) node = node.parentNode;
nodeType here is undefined and event.type is "unload". Reading furhter up the code it seems event.target is undefined as well. Does the event "unload" even have a target?
This needs further patching it seems.
Please let me know if you need further input/trace/etc...
Comments and changes to this ticket
-
splintor October 15th, 2008 @ 07:55 AM
- Tag changed from “bug, events, prototype” to “bug, events, prototype”
See http://extjs.com/forum/showthrea... where I first reported this problem about a year ago with prototype 1.6, and provided a patch.
It is strange as I don't see anything special about Ext here - it seems every app that will register the unload event will run into this problem. Yet, no one seems to notice it...
-
skaue October 15th, 2008 @ 08:07 AM
Well, Firefox just doesn't care. It unloads and doesn't haul at me like IE does. Before I lost my mind while waiting for a stable patch, we just unplugged the adapter in ExtJs that uses prototype and use Ext base adapter. The error is gone now. It feels easy to blame the adapter, but I'm still under the impression that the failure lays in the Event-object of prototype.
In our project we're actually planning to move away from prototype and that feels sad. I've been a fanboy of prototype several years now. Seems jQuery is winning the battlefield here at the office... :-/
-
DHH October 30th, 2008 @ 11:52 AM
- State changed from “new” to “invalid”
Prototype now has it's own ticket tracker at http://prototype.lighthouseapp.c...
-
-
David Reese November 11th, 2008 @ 04:47 PM
= "its own ticket tracker". http://www.37signals.com/svn/pos...
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
DHH
splintor