From ebdf9110fdd23c67308879d04dbda2596552ebb4 Mon Sep 17 00:00:00 2001 From: Phoenix Eliot Date: Mon, 30 Mar 2020 21:54:41 -0600 Subject: [PATCH] Fix order of operations for tooltipPlacement (#22772) tooltipPlacement was broken by https://github.com/ant-design/ant-design/commit/a6c207d9dd9ea14c251d57b186b710b3066a7a64. The grouping of this term was getting parsed as `(tooltipPlacement || vertical) ? 'right' : 'top'`, which was incorrect. --- components/slider/index.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/components/slider/index.tsx b/components/slider/index.tsx index d0d905d862..21cadb7f55 100644 --- a/components/slider/index.tsx +++ b/components/slider/index.tsx @@ -106,7 +106,7 @@ export default class Slider extends React.Component { prefixCls={tooltipPrefixCls} title={tipFormatter ? tipFormatter(value) : ''} visible={visible} - placement={tooltipPlacement || vertical ? 'right' : 'top'} + placement={tooltipPlacement || (vertical ? 'right' : 'top')} transitionName="zoom-down" key={index} overlayClassName={`${prefixCls}-tooltip`}