fix: slider: ensure tooltip is visually hidden when slider is disabled (

#651)
EightfoldAI · Jun 23, 2023 · b810ba8 · b810ba8
1 parent 565581b
commit b810ba8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/src/components/Slider/Slider.tsx b/src/components/Slider/Slider.tsx
@@ -841,6 +841,9 @@ export const Slider: FC<SliderProps> = React.forwardRef(
                     styles.sliderTooltip,
                     tooltipProps?.classNames,
                   ])}
+                  wrapperClassNames={
+                    mergedDisabled ? styles.hideSliderTooltip : ''
+                  }
                   closeOnReferenceClick={false}
                   content={getTooltipContentByValue(val)}
                   key={`value-tooltip-${index}`}

diff --git a/src/components/Slider/slider.module.scss b/src/components/Slider/slider.module.scss
@@ -136,6 +136,11 @@ $small-min-label-offset-with-steps: 0;
     &-tooltip {
       min-width: max-content;
     }
+
+    .hide-slider-tooltip {
+      opacity: 0;
+      visibility: hidden;
+    }
   }
 
   .rail-marker-segments {