You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pivot.apache.org by rw...@apache.org on 2014/06/12 18:58:59 UTC

svn commit: r1602230 - /pivot/branches/2.0.x/wtk/src/org/apache/pivot/wtk/Container.java

Author: rwhitcomb
Date: Thu Jun 12 16:58:59 2014
New Revision: 1602230

URL: http://svn.apache.org/r1602230
Log:
PIVOT-951: Only first click is received by container listeners

Only the first (ever) click into a container is received by listeners
because the "mouseClickConsumed" variable never gets reset after the
first click has been detected.  This change simply resets the variable
on a mouse up event, which enables all subsequent clicks to be
received as well.

It is noted that perhaps this variable is not really necessary and
could be removed. Will investigate further.


Modified:
    pivot/branches/2.0.x/wtk/src/org/apache/pivot/wtk/Container.java

Modified: pivot/branches/2.0.x/wtk/src/org/apache/pivot/wtk/Container.java
URL: http://svn.apache.org/viewvc/pivot/branches/2.0.x/wtk/src/org/apache/pivot/wtk/Container.java?rev=1602230&r1=1602229&r2=1602230&view=diff
==============================================================================
--- pivot/branches/2.0.x/wtk/src/org/apache/pivot/wtk/Container.java (original)
+++ pivot/branches/2.0.x/wtk/src/org/apache/pivot/wtk/Container.java Thu Jun 12 16:58:59 2014
@@ -783,6 +783,7 @@ public abstract class Container extends 
     @Override
     protected boolean mouseUp(Mouse.Button button, int x, int y) {
         boolean consumed = false;
+        mouseClickConsumed = false;
 
         if (isEnabled()) {
             // Notify container listeners