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);