Fixed layout validation so only the invalid dimension is set to undefined

svn changeset:6173/svn branch:trunk

1 files changed, 15 insertions, 12 deletions

diff --git a/src/com/itmill/toolkit/ui/AbstractComponent.java b/src/com/itmill/toolkit/ui/AbstractComponent.java
index 48717c4bc2..5a692736e7 100644
--- a/src/com/itmill/toolkit/ui/AbstractComponent.java
+++ b/src/com/itmill/toolkit/ui/AbstractComponent.java
@@ -609,20 +609,23 @@ public abstract class AbstractComponent implements Component, MethodEventSource
 
             // Only paint content of visible components.
             if (isVisible()) {
+                boolean validHeight = true;
+                boolean validWidth = true;
 
-                // TODO split this method
-                if (getApplication() != null
-                        && getApplication().isDebugMode()
-                        && !DebugUtilities.validateComponentRelativeSizes(this,
-                                false)) {
+                if (getApplication() != null && getApplication().isDebugMode()) {
+                    validHeight = DebugUtilities.checkHeights(this);
+                    validWidth = DebugUtilities.checkWidths(this);
+                }
+                if (!validHeight || !validWidth) {
                     addStyleName("invalidlayout");
-                } else {
-                    if (getHeight() >= 0) {
-                        target.addAttribute("height", "" + getCSSHeight());
-                    }
-                    if (getWidth() >= 0) {
-                        target.addAttribute("width", "" + getCSSWidth());
-                    }
+                }
+
+                if (validHeight && getHeight() >= 0) {
+                    target.addAttribute("height", "" + getCSSHeight());
+                }
+
+                if (validWidth && getWidth() >= 0) {
+                    target.addAttribute("width", "" + getCSSWidth());
                 }
                 if (styles != null && styles.size() > 0) {
                     target.addAttribute("style", getStyle());