Merge pull request #1451 from KapJI/slider-fix

Fix Slider issue #1448
mui · Aug 21, 2015 · ed7e830 · ed7e830
2 parents 5a2de1a + 1474688
commit ed7e830
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/src/ripples/focus-ripple.jsx b/src/ripples/focus-ripple.jsx
@@ -113,8 +113,13 @@ const FocusRipple = React.createClass({
     const width = el.offsetWidth;
     const size = Math.max(height, width);
 
+    let oldTop = 0;
+    // For browsers that don't support endsWith()
+    if (el.style.top.indexOf('px', el.style.top.length - 2) !== -1) {
+      oldTop = parseInt(el.style.top);
+    } 
     el.style.height = size + 'px';
-    el.style.top = (size / 2 * -1) + (height / 2) + 'px';
+    el.style.top = (height / 2) - (size / 2 ) + oldTop + 'px';
   },
 
 });

diff --git a/src/slider.jsx b/src/slider.jsx
@@ -246,7 +246,7 @@ let Slider = React.createClass({
       this.props.disabled && styles.handleWhenDisabled
     );
 
-    let rippleStyle = {height: '12px', width: '12px'};
+    let rippleStyle = {height: '12px', width: '12px', overflow: 'visible'};
 
     if ((this.state.hovered || this.state.focused) && !this.props.disabled) {
       remainingStyles.backgroundColor = this.getTheme().trackColorSelected;