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 2010/08/02 14:22:31 UTC

svn commit: r981481 - in /pivot/branches/1.5.x: RELEASE-NOTES wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraFormSkin.java

Author: gbrown
Date: Mon Aug  2 12:22:30 2010
New Revision: 981481

URL: http://svn.apache.org/viewvc?rev=981481&view=rev
Log:
Resolve PIVOT-582 for Pivot 1.5.x.

Modified:
    pivot/branches/1.5.x/RELEASE-NOTES
    pivot/branches/1.5.x/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraFormSkin.java

Modified: pivot/branches/1.5.x/RELEASE-NOTES
URL: http://svn.apache.org/viewvc/pivot/branches/1.5.x/RELEASE-NOTES?rev=981481&r1=981480&r2=981481&view=diff
==============================================================================
--- pivot/branches/1.5.x/RELEASE-NOTES (original)
+++ pivot/branches/1.5.x/RELEASE-NOTES Mon Aug  2 12:22:30 2010
@@ -36,4 +36,4 @@ Release Notes for Pivot 1.5.1:
     * [PIVOT-518] - Serialization of "structs" fails
     * [PIVOT-519] - CSVSerializer does not trim values
     * [PIVOT-533] - org.apache.pivot.collections.Sequence.Tree - add, get, insert, remove & update methods do not handle empty Paths
-
+    * [PIVOT-582] - IllegalArgumentException : width is negative when using Form Heading

Modified: pivot/branches/1.5.x/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraFormSkin.java
URL: http://svn.apache.org/viewvc/pivot/branches/1.5.x/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraFormSkin.java?rev=981481&r1=981480&r2=981481&view=diff
==============================================================================
--- pivot/branches/1.5.x/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraFormSkin.java (original)
+++ pivot/branches/1.5.x/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraFormSkin.java Mon Aug  2 12:22:30 2010
@@ -729,7 +729,7 @@ public class TerraFormSkin extends Conta
                 && !showFirstSectionHeading) {
                 separator.setVisible(false);
             } else {
-                int separatorWidth = width - (padding.left + padding.right);
+                int separatorWidth = Math.max(width - (padding.left + padding.right), 0);
                 separator.setVisible(true);
                 separator.setSize(separatorWidth, separator.getPreferredHeight(separatorWidth));
                 separator.setLocation(padding.left, rowY);