From: Artur Signell <artur@vaadin.com>
Date: Wed, 20 Aug 2014 11:18:58 +0000 (+0300)
Subject: Do not hide handle as it causes IE8 problems (#14412)
X-Git-Tag: 7.4.0.beta1~311
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=0110af79f162c5c58e15d20bc35d8655fc5a8cde;p=vaadin-framework.git

Do not hide handle as it causes IE8 problems (#14412)

Change-Id: I2b981c00bf1ef3b9e1e0ccd1cdd3678016b11d14
---

diff --git a/client/src/com/vaadin/client/ui/VSlider.java b/client/src/com/vaadin/client/ui/VSlider.java
index bc314e0314..86e5ef129d 100644
--- a/client/src/com/vaadin/client/ui/VSlider.java
+++ b/client/src/com/vaadin/client/ui/VSlider.java
@@ -22,7 +22,6 @@ import com.google.gwt.dom.client.Element;
 import com.google.gwt.dom.client.Style.Display;
 import com.google.gwt.dom.client.Style.Overflow;
 import com.google.gwt.dom.client.Style.Unit;
-import com.google.gwt.dom.client.Style.Visibility;
 import com.google.gwt.event.dom.client.KeyCodes;
 import com.google.gwt.event.logical.shared.ValueChangeEvent;
 import com.google.gwt.event.logical.shared.ValueChangeHandler;
@@ -122,7 +121,6 @@ public class VSlider extends SimpleFocusablePanel implements Field,
         // Hide initially
         smaller.getStyle().setDisplay(Display.NONE);
         bigger.getStyle().setDisplay(Display.NONE);
-        handle.getStyle().setVisibility(Visibility.HIDDEN);
 
         sinkEvents(Event.MOUSEEVENTS | Event.ONMOUSEWHEEL | Event.KEYEVENTS
                 | Event.FOCUSEVENTS | Event.TOUCHEVENTS);
@@ -254,9 +252,6 @@ public class VSlider extends SimpleFocusablePanel implements Field,
 
         // clear unnecessary opposite handle attribute
         handle.getStyle().clearProperty(oppositeHandleAttribute);
-
-        // Restore visibility
-        handle.getStyle().setVisibility(Visibility.VISIBLE);
     }
 
     @Override