Side navigation
#3015 closed enhancement (fixed)
Opened June 10, 2008 03:02PM UTC
Closed June 22, 2010 02:41AM UTC
Last modified March 14, 2012 04:51PM UTC
Restrict onunload removal of eventhandlers to IE and Firefox 2
Reported by: | JeroenH | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 1.3 |
Component: | event | Version: | 1.2.6 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
In event.js the last lines remove registered eventhandlers on unload in all browsers. This is needed for IE's memory leaks and initially only targeted IE. In #1911 this was changed to apply to all browsers, but the only other browser in need of a fix there was Firefox 2.
This change breaks fast history navigation in Opera (and also in Firefox).
When testing the testcase in #1911, I can reproduce the problem in FF2, but it does not appear anymore in FF3.
It would be better to restrict this change to the browsers that need the fix:
// Prevent memory leaks in IE // And prevent errors on refresh with events like mouseover in mozilla < 1.9 // Window isn't included so as not to unbind existing unload events if (jQuery.browser.msie || (jQuery.browser.mozilla && parseFloat(jQuery.browser.version) < 1.9) ){ jQuery(window).bind("unload", function() { jQuery("*").add(document).unbind(); }); }
Attachments (0)
Change History (5)
Changed June 10, 2008 08:16PM UTC by comment:1
component: | core → event |
---|---|
owner: | → brandon |
type: | bug → enhancement |
Changed April 22, 2009 01:24AM UTC by comment:2
owner: | brandon |
---|
Changed September 22, 2009 08:02AM UTC by comment:3
This got fixed somewhere, by adding conditional comments for IE.
Changed October 17, 2009 03:23PM UTC by comment:4
This was fixed in ticket:5198 (changeset:6551).
Changed June 22, 2010 02:41AM UTC by comment:5
resolution: | → fixed |
---|---|
status: | new → closed |