commit d97c97df4ca51b72acda6b9041ceadf854e52ea4
author ecc@google.com <ecc@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Fri Jan 23 17:45:22 2009 +0000
committer ecc@google.com <ecc@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Fri Jan 23 17:45:22 2009 +0000
tree 2fb954784f3b5df016daa2fd25e12b2383f68c67
parent 8ffa2aa0cbeb7c7918e0ef5674ffa95580078b0f

Fixed issue 3307 by not gating the selection of an item on whether it is focusable
Review by:jlabanca

git-svn-id: https://google-web-toolkit.googlecode.com/svn/releases/1.6@4521 8db76d5a-ed1c-0410-87a9-c151d255dfc7

user/src/com/google/gwt/user/client/ui/Tree.java

diff --git a/user/src/com/google/gwt/user/client/ui/Tree.java b/user/src/com/google/gwt/user/client/ui/Tree.java
index 9dff280..e6b5a07 100644
--- a/user/src/com/google/gwt/user/client/ui/Tree.java
+++ b/user/src/com/google/gwt/user/client/ui/Tree.java
@@ -929,7 +929,7 @@
     DOM.setIntStyleAttribute(focusable, "zIndex", -1);
     DOM.appendChild(getElement(), focusable);
 
-    sinkEvents(Event.MOUSEEVENTS | Event.ONCLICK | Event.KEYEVENTS);
+    sinkEvents(Event.ONMOUSEDOWN | Event.ONCLICK | Event.KEYEVENTS);
     DOM.sinkEvents(focusable, Event.FOCUSEVENTS);
 
     // The 'root' item is invisible and serves only as a container
@@ -1154,9 +1154,10 @@
     }
     curSelection = item;
 
-    if (moveFocus && curSelection != null) {
-      moveFocus();
-
+    if (curSelection != null) {
+      if (moveFocus) {
+        moveFocus();
+      }
       // Select the item and fire the selection event.
       curSelection.setSelected(true);
       if (fireEvents) {