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 369: getEventTargetType() does not recognize the target type if the element has additional classes
2 people starred this issue and may be notified of changes. Back to list
Status:  Fixed
Owner:  ----
Closed:  Sep 2013


 
Reported by Pami.Ketolainen, Nov 2, 2012
What steps will reproduce the problem?
1. Add some other class to dynatree-title node, for example in custom rendering.
2. node.getEventTargetType(event) will return null, when for example, clicking on the node title


This is because getEventTargetType treats target.className as single class name, when it actually can contain several.

I've attached a quick patch to make it support multiple classes, but it's not very efficient approach.

Maybe reverting the tree.options.classNames to make a class name => type map and then iterating over target.className.split(/\s+/) to get the type would be better and cleaner.
getEventTargetType_classes.patch
1.1 KB   View   Download
Nov 2, 2012
Project Member #1 moo...@wwwendt.de
(No comment was entered for this change.)
Status: Accepted
Labels: Milestone-Release1.2.3
Feb 10, 2013
Project Member #2 moo...@wwwendt.de
(No comment was entered for this change.)
Labels: -Milestone-Release1.2.3 Milestone-Release1.2.4
Feb 12, 2013
Project Member #3 moo...@wwwendt.de
1.2.4 was a hotfix
Labels: -Milestone-Release1.2.4 Milestone-Release1.2.5
Sep 8, 2013
Project Member #4 moo...@wwwendt.de
This issue was closed by revision r678.
Status: Fixed
Sep 8, 2013
Project Member #5 moo...@wwwendt.de
 Issue 421  has been merged into this issue.

Powered by Google Project Hosting