Merge pull request #1852 from Huuums/feature/brush-always-show-text-s…

…upport Always show text support for Brush
recharts · Oct 16, 2019 · 422941a · 422941a
2 parents c805aae + 3b82711
commit 422941a
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/src/cartesian/Brush.js b/src/cartesian/Brush.js
@@ -45,6 +45,7 @@ class Brush extends PureComponent {
     onChange: PropTypes.func,
     updateId: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
     leaveTimeOut: PropTypes.number,
+    alwaysShowText: PropTypes.bool,
   };
 
   static defaultProps = {
@@ -55,6 +56,7 @@ class Brush extends PureComponent {
     stroke: '#666',
     padding: { top: 1, right: 1, bottom: 1, left: 1 },
     leaveTimeOut: 1000,
+    alwaysShowText: false
   };
 
   constructor(props) {
@@ -427,7 +429,7 @@ class Brush extends PureComponent {
   }
 
   render() {
-    const { data, className, children, x, y, width, height } = this.props;
+    const { data, className, children, x, y, width, height, alwaysShowText } = this.props;
     const { startX, endX, isTextActive, isSlideMoving, isTravellerMoving } = this.state;
 
     if (!data || !data.length || !isNumber(x) || !isNumber(y) || !isNumber(width) ||
@@ -452,7 +454,7 @@ class Brush extends PureComponent {
         {this.renderSlide(startX, endX)}
         {this.renderTraveller(startX, 'startX')}
         {this.renderTraveller(endX, 'endX')}
-        {(isTextActive || isSlideMoving || isTravellerMoving) && this.renderText()}
+        {(isTextActive || isSlideMoving || isTravellerMoving || alwaysShowText) && this.renderText()}
       </Layer>
     );
   }

diff --git a/test/specs/cartesian/BrushSpec.js b/test/specs/cartesian/BrushSpec.js
@@ -76,4 +76,11 @@ describe('<Brush />', () => {
     wrapper.simulate('mouseMove');
     wrapper.simulate('mouseLeave');
   });
+
+  it('render text when alwaysShowText is true', () => {
+    const wrapper = mount(
+      <Brush x={100} y={50} width={400} height={40} data={data} alwaysShowText />
+    );
+    expect(wrapper.find('.recharts-layer.recharts-brush-texts').length).to.equal(1);
+  });
 });