You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pivot.apache.org by gb...@apache.org on 2009/06/30 01:29:18 UTC

svn commit: r789492 - in /incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin: ButtonSkin.java terra/TerraMenuButtonSkin.java

Author: gbrown
Date: Mon Jun 29 23:29:17 2009
New Revision: 789492

URL: http://svn.apache.org/viewvc?rev=789492&view=rev
Log:
Return focus to toolbar-style menu button when Escape is pressed in the menu popup.

Modified:
    incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/ButtonSkin.java
    incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/terra/TerraMenuButtonSkin.java

Modified: incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/ButtonSkin.java
URL: http://svn.apache.org/viewvc/incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/ButtonSkin.java?rev=789492&r1=789491&r2=789492&view=diff
==============================================================================
--- incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/ButtonSkin.java (original)
+++ incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/ButtonSkin.java Mon Jun 29 23:29:17 2009
@@ -74,7 +74,7 @@
     public void focusedChanged(Component component, boolean temporary) {
         super.focusedChanged(component, temporary);
 
-        repaintComponent(!component.isFocused());
+        repaintComponent();
     }
 
     // Component mouse events

Modified: incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/terra/TerraMenuButtonSkin.java
URL: http://svn.apache.org/viewvc/incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/terra/TerraMenuButtonSkin.java?rev=789492&r1=789491&r2=789492&view=diff
==============================================================================
--- incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/terra/TerraMenuButtonSkin.java (original)
+++ incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/terra/TerraMenuButtonSkin.java Mon Jun 29 23:29:17 2009
@@ -231,8 +231,7 @@
 
     @Override
     public boolean isFocusable() {
-        return !toolbar
-            || menuPopup.isClosed();
+        return true;
     }
 
     public Font getFont() {