diff options
author | Artur Signell <artur@vaadin.com> | 2015-07-11 21:54:02 +0300 |
---|---|---|
committer | Vaadin Code Review <review@vaadin.com> | 2015-08-12 08:28:15 +0000 |
commit | 2b19ef308931a0f0d2d1a87b8ac11a38412eb0ef (patch) | |
tree | 8acbc6ea89cf8b02a3e3221d8fb0d4449074ff8d | |
parent | 53c114ec5e0c2223dfb6e9456c60a05e1c2ad381 (diff) | |
download | vaadin-framework-2b19ef308931a0f0d2d1a87b8ac11a38412eb0ef.tar.gz vaadin-framework-2b19ef308931a0f0d2d1a87b8ac11a38412eb0ef.zip |
Do not force zero height for empty caption height in FormLayout (#15409)
Change-Id: I0575556785aa2c5cedb8ec40e2db1ec6644967cc
-rw-r--r-- | client/src/com/vaadin/client/ui/VFormLayout.java | 17 |
1 files changed, 0 insertions, 17 deletions
diff --git a/client/src/com/vaadin/client/ui/VFormLayout.java b/client/src/com/vaadin/client/ui/VFormLayout.java index 84a9b4f3dd..bcbb3ebf7b 100644 --- a/client/src/com/vaadin/client/ui/VFormLayout.java +++ b/client/src/com/vaadin/client/ui/VFormLayout.java @@ -22,7 +22,6 @@ import java.util.List; import com.google.gwt.aria.client.Roles; import com.google.gwt.dom.client.Element; -import com.google.gwt.dom.client.Style.Overflow; import com.google.gwt.event.dom.client.ClickEvent; import com.google.gwt.event.dom.client.ClickHandler; import com.google.gwt.user.client.DOM; @@ -327,22 +326,6 @@ public class VFormLayout extends SimplePanel { requiredFieldIndicator = null; } } - - // Workaround for IE weirdness, sometimes returns bad height in some - // circumstances when Caption is empty. See #1444 - // IE7 bugs more often. I wonder what happens when IE8 arrives... - // FIXME: This could be unnecessary for IE8+ - if (BrowserInfo.get().isIE()) { - if (isEmpty) { - setHeight("0px"); - getElement().getStyle().setOverflow(Overflow.HIDDEN); - } else { - setHeight(""); - getElement().getStyle().clearOverflow(); - } - - } - } /** |