You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pivot.apache.org by sm...@apache.org on 2013/03/12 13:32:02 UTC

svn commit: r1455508 - /pivot/branches/2.0.x/wtk/src/org/apache/pivot/wtk/content/ButtonDataRenderer.java

Author: smartini
Date: Tue Mar 12 12:32:02 2013
New Revision: 1455508

URL: http://svn.apache.org/r1455508
Log:
PIVOT-898 and PIVOT-901, a fix to solve both cases (and without add a new style)

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

Modified: pivot/branches/2.0.x/wtk/src/org/apache/pivot/wtk/content/ButtonDataRenderer.java
URL: http://svn.apache.org/viewvc/pivot/branches/2.0.x/wtk/src/org/apache/pivot/wtk/content/ButtonDataRenderer.java?rev=1455508&r1=1455507&r2=1455508&view=diff
==============================================================================
--- pivot/branches/2.0.x/wtk/src/org/apache/pivot/wtk/content/ButtonDataRenderer.java (original)
+++ pivot/branches/2.0.x/wtk/src/org/apache/pivot/wtk/content/ButtonDataRenderer.java Tue Mar 12 12:32:02 2013
@@ -98,7 +98,7 @@ public class ButtonDataRenderer extends 
         // Update the label
         label.setText(text != null ? text : "");
 
-        if (text == null) {
+        if (text == null || (text.length() == 0 && getFillIcon())) {
             label.setVisible(false);
         } else {
             label.setVisible(true);