| Issue 284: | Focus Events Have Memory Leak | |
| 1 person starred this issue and may be notified of changes. | Back to list |
What steps will reproduce the problem?
1. Add console.log("focus") to start of _onFocus function
2. Click between 2 nodes in dynatree
3. Notice the number of log messages increases each time you change focus
Detected in version 1.2.0 in Chrome (win7)
To fix, update the code immediately after the __focusHandler function as follows:
var div = this.tree.divTree;
if (!this.hasBound) {
if ( div.addEventListener ) {
div.addEventListener("focus", __focusHandler, true);
div.addEventListener("blur", __focusHandler, true);
} else {
div.onfocusin = div.onfocusout = __focusHandler;
}
this.hasBound = true;
}
Jun 9, 2012
This would mean that bind() was called on every click. I tried it with a sample from the demos, and it looked ok. Could you send a test file, that reproduces it?
Status:
Waiting
Jun 16, 2012
(No comment was entered for this change.)
Labels:
-Milestone-Release1.2.1 Milestone-Release1.2.2
Oct 6, 2012
(No comment was entered for this change.)
Labels:
-Milestone-Release1.2.2 Milestone-Release1.2.x
Oct 6, 2013
Closing 'Waiting' issues without timely answer
Status:
Done
May 3, 2014
(No comment was entered for this change.)
Status:
WontFix
Labels: -Milestone-Release1.2.x Milestone-Release1.2.5 |
Labels: Milestone-Release1.2.1