My favorites | Sign in
Project Home Downloads
READ-ONLY: This project has been archived. For more information see this post.
Search
for
  Advanced search   Search tips   Subscriptions
Issue 284: Focus Events Have Memory Leak
1 person starred this issue and may be notified of changes. Back to list
Status:  WontFix
Owner:  ----
Closed:  Oct 2013


 
Reported by peter.do...@ddrit.com, Feb 21, 2012
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;
}
Feb 21, 2012
Project Member #1 moo...@wwwendt.de
(No comment was entered for this change.)
Status: Accepted
Labels: Milestone-Release1.2.1
Jun 9, 2012
Project Member #2 moo...@wwwendt.de
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
Project Member #3 moo...@wwwendt.de
(No comment was entered for this change.)
Labels: -Milestone-Release1.2.1 Milestone-Release1.2.2
Oct 6, 2012
Project Member #4 moo...@wwwendt.de
(No comment was entered for this change.)
Labels: -Milestone-Release1.2.2 Milestone-Release1.2.x
Oct 6, 2013
Project Member #5 moo...@wwwendt.de
Closing 'Waiting' issues without timely answer
Status: Done
May 3, 2014
Project Member #6 moo...@wwwendt.de
(No comment was entered for this change.)
Status: WontFix
Labels: -Milestone-Release1.2.x Milestone-Release1.2.5

Powered by Google Project Hosting