rsuite 4.10.3 → 4.10.4

package/es/Cascader/Cascader.js

@@ -36,6 +36,7 @@ function (_React$Component) {
     _this.triggerRef = void 0;
     _this.containerRef = void 0;
     _this.positionRef = void 0;
+    _this.menuWrapperRef = void 0;
     _this.menuContainerRef = void 0;
     _this.isControlled = void 0;
 
@@ -291,7 +292,7 @@ function (_React$Component) {
       items: []
     };
     _this.state = _extends({}, initState, {}, getDerivedStateForCascade(props, initState), {
-      flattenData: flattenTree(props.data)
+      flattenData: flattenTree(props.data, props.childrenKey)
     });
     _this.isControlled = !_isUndefined(props.value);
     _this.triggerRef = React.createRef();
@@ -299,6 +300,7 @@ function (_React$Component) {
     _this.positionRef = React.createRef();
     _this.toggleRef = React.createRef(); // for test
 
+    _this.menuWrapperRef = React.createRef();
     _this.menuContainerRef = React.createRef();
     return _this;
   }
@@ -307,7 +309,8 @@ function (_React$Component) {
     var value = nextProps.value,
         data = nextProps.data,
         labelKey = nextProps.labelKey,
-        valueKey = nextProps.valueKey;
+        valueKey = nextProps.valueKey,
+        childrenKey = nextProps.childrenKey;
 
     if (data !== prevState.data) {
       var _prevState$selectNode;
@@ -325,12 +328,12 @@ function (_React$Component) {
           return _stringToObject(item, labelKey, valueKey);
         })), {
           data: data,
-          flattenData: flattenTree(data)
+          flattenData: flattenTree(data, childrenKey)
         });
       }
 
       return _extends({}, getDerivedStateForCascade(nextProps, prevState), {
-        flattenData: flattenTree(data),
+        flattenData: flattenTree(data, childrenKey),
         data: data
       });
     }
@@ -441,6 +444,7 @@ function (_React$Component) {
     var menuProps = _pick(this.props, Object.keys(_omit(dropdownMenuPropTypes, ['classPrefix'])));
 
     return React.createElement(MenuWrapper, {
+      ref: this.menuWrapperRef,
       className: classes,
       style: menuStyle,
       getToggleInstance: this.getToggleInstance,

package/es/Slider/Slider.d.ts

@@ -1,7 +1,9 @@
 import * as React from 'react';
-import { StandardProps } from '../@types/common';
+import { StandardProps, FormControlBaseProps } from '../@types/common';
 
-export interface SliderProps<ValueType = number> extends StandardProps {
+export interface SliderProps<ValueType = number>
+  extends StandardProps,
+    FormControlBaseProps<ValueType> {
   /** Minimum value of sliding range */
   min?: number;
 
@@ -11,12 +13,6 @@ export interface SliderProps<ValueType = number> extends StandardProps {
   /** Slide the value of one step */
   step?: number;
 
-  /** Value (Controlled) */
-  value?: ValueType;
-
-  /** Default value */
-  defaultValue?: ValueType;
-
   /** A css class to apply to the Handle node. */
   handleClassName?: string;
 
@@ -44,9 +40,6 @@ export interface SliderProps<ValueType = number> extends StandardProps {
   /** Vertical Slide */
   vertical?: boolean;
 
-  /** Callback function that changes data */
-  onChange?: (value: ValueType, event: React.MouseEvent) => void;
-
   /** Customize labels on the render ruler */
   renderMark?: (mark: number) => React.ReactNode;
 }

package/lib/Cascader/Cascader.js

@@ -66,6 +66,7 @@ function (_React$Component) {
     _this.triggerRef = void 0;
     _this.containerRef = void 0;
     _this.positionRef = void 0;
+    _this.menuWrapperRef = void 0;
     _this.menuContainerRef = void 0;
     _this.isControlled = void 0;
 
@@ -321,7 +322,7 @@ function (_React$Component) {
       items: []
     };
     _this.state = (0, _extends3.default)({}, initState, {}, (0, _utils.getDerivedStateForCascade)(props, initState), {
-      flattenData: (0, _treeUtils.flattenTree)(props.data)
+      flattenData: (0, _treeUtils.flattenTree)(props.data, props.childrenKey)
     });
     _this.isControlled = !(0, _isUndefined2.default)(props.value);
     _this.triggerRef = React.createRef();
@@ -329,6 +330,7 @@ function (_React$Component) {
     _this.positionRef = React.createRef();
     _this.toggleRef = React.createRef(); // for test
 
+    _this.menuWrapperRef = React.createRef();
     _this.menuContainerRef = React.createRef();
     return _this;
   }
@@ -337,7 +339,8 @@ function (_React$Component) {
     var value = nextProps.value,
         data = nextProps.data,
         labelKey = nextProps.labelKey,
-        valueKey = nextProps.valueKey;
+        valueKey = nextProps.valueKey,
+        childrenKey = nextProps.childrenKey;
 
     if (data !== prevState.data) {
       var _prevState$selectNode;
@@ -355,12 +358,12 @@ function (_React$Component) {
           return (0, _stringToObject2.default)(item, labelKey, valueKey);
         })), {
           data: data,
-          flattenData: (0, _treeUtils.flattenTree)(data)
+          flattenData: (0, _treeUtils.flattenTree)(data, childrenKey)
         });
       }
 
       return (0, _extends3.default)({}, (0, _utils.getDerivedStateForCascade)(nextProps, prevState), {
-        flattenData: (0, _treeUtils.flattenTree)(data),
+        flattenData: (0, _treeUtils.flattenTree)(data, childrenKey),
         data: data
       });
     }
@@ -469,6 +472,7 @@ function (_React$Component) {
     var classes = (0, _classnames.default)(this.addPrefix('cascader-menu'), menuClassName, (_classNames2 = {}, _classNames2[this.addPrefix('inline')] = inline, _classNames2));
     var menuProps = (0, _pick2.default)(this.props, Object.keys((0, _omit2.default)(_DropdownMenu.dropdownMenuPropTypes, ['classPrefix'])));
     return React.createElement(_Picker.MenuWrapper, {
+      ref: this.menuWrapperRef,
       className: classes,
       style: menuStyle,
       getToggleInstance: this.getToggleInstance,

package/lib/Slider/Slider.d.ts

@@ -1,7 +1,9 @@
 import * as React from 'react';
-import { StandardProps } from '../@types/common';
+import { StandardProps, FormControlBaseProps } from '../@types/common';
 
-export interface SliderProps<ValueType = number> extends StandardProps {
+export interface SliderProps<ValueType = number>
+  extends StandardProps,
+    FormControlBaseProps<ValueType> {
   /** Minimum value of sliding range */
   min?: number;
 
@@ -11,12 +13,6 @@ export interface SliderProps<ValueType = number> extends StandardProps {
   /** Slide the value of one step */
   step?: number;
 
-  /** Value (Controlled) */
-  value?: ValueType;
-
-  /** Default value */
-  defaultValue?: ValueType;
-
   /** A css class to apply to the Handle node. */
   handleClassName?: string;
 
@@ -44,9 +40,6 @@ export interface SliderProps<ValueType = number> extends StandardProps {
   /** Vertical Slide */
   vertical?: boolean;
 
-  /** Callback function that changes data */
-  onChange?: (value: ValueType, event: React.MouseEvent) => void;
-
   /** Customize labels on the render ruler */
   renderMark?: (mark: number) => React.ReactNode;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rsuite",
-  "version": "4.10.3",
+  "version": "4.10.4",
   "description": "A suite of react components",
   "main": "lib/index.js",
   "typings": "lib/index.d.ts",
@@ -29,7 +29,8 @@
     "test:component": "cross-env NODE_ENV=coverage RUN_ENV=test karma start --single-run",
     "test:styles": "node test/stylesSpec.js",
     "coveralls": "cat ./coverage/lcov/lcov.info | ./node_modules/.bin/coveralls",
-    "prepublishOnly": "npm run build"
+    "prepublishOnly": "npm run build",
+    "version": "npm run changelog && git add -A"
   },
   "keywords": [
     "react",

package/src/Cascader/Cascader.tsx

@@ -73,6 +73,7 @@ class Cascader extends React.Component<CascaderProps, CascaderState> {
   triggerRef: React.RefObject<any>;
   containerRef: React.RefObject<any>;
   positionRef: React.RefObject<any>;
+  menuWrapperRef: React.RefObject<any>;
   menuContainerRef: React.RefObject<any>;
   isControlled: boolean;
 
@@ -98,7 +99,7 @@ class Cascader extends React.Component<CascaderProps, CascaderState> {
     this.state = {
       ...initState,
       ...getDerivedStateForCascade(props, initState),
-      flattenData: flattenTree(props.data)
+      flattenData: flattenTree(props.data, props.childrenKey)
     };
 
     this.isControlled = !_.isUndefined(props.value);
@@ -108,11 +109,12 @@ class Cascader extends React.Component<CascaderProps, CascaderState> {
     this.toggleRef = React.createRef();
 
     // for test
+    this.menuWrapperRef = React.createRef();
     this.menuContainerRef = React.createRef();
   }
 
   static getDerivedStateFromProps(nextProps, prevState) {
-    const { value, data, labelKey, valueKey } = nextProps;
+    const { value, data, labelKey, valueKey, childrenKey } = nextProps;
     if (data !== prevState.data) {
       // First get the value of the clicked node `selectNodeValue`, and then get the new `newChildren`.
       const selectNodeValue = prevState?.selectNode?.[valueKey];
@@ -129,13 +131,13 @@ class Cascader extends React.Component<CascaderProps, CascaderState> {
             newChildren.map(item => stringToObject(item, labelKey, valueKey))
           ),
           data,
-          flattenData: flattenTree(data)
+          flattenData: flattenTree(data, childrenKey)
         };
       }
 
       return {
         ...getDerivedStateForCascade(nextProps, prevState),
-        flattenData: flattenTree(data),
+        flattenData: flattenTree(data, childrenKey),
         data
       };
     }
@@ -450,6 +452,7 @@ class Cascader extends React.Component<CascaderProps, CascaderState> {
 
     return (
       <MenuWrapper
+        ref={this.menuWrapperRef}
         className={classes}
         style={menuStyle}
         getToggleInstance={this.getToggleInstance}

package/src/Cascader/test/CascaderSpec.js

@@ -1,5 +1,5 @@
 import React from 'react';
-import ReactTestUtils from 'react-dom/test-utils';
+import ReactTestUtils, { act } from 'react-dom/test-utils';
 import Cascader from '../Cascader';
 import Button from '../../Button';
 import { getDOMNode, getInstance } from '@test/testUtils';
@@ -214,4 +214,59 @@ describe('Cascader', () => {
     assert.equal(instance2.querySelector('.rs-picker-toggle-placeholder').innerText, 'Select');
     assert.equal(instance3.querySelector('.rs-picker-toggle-placeholder').innerText, 'Select');
   });
+
+  it('Should show search items with childrenKey', () => {
+    const itemsWithChildrenKey = {
+      childrenKey: 'sub',
+      data: [
+        {
+          value: 't',
+          label: 't'
+        },
+        {
+          value: 'h',
+          label: 'h'
+        },
+        {
+          value: 'g',
+          label: 'g',
+          sub: [
+            {
+              value: 'g-m',
+              label: 'g-m'
+            },
+            {
+              value: 'g-b',
+              label: 'g-b'
+            }
+          ]
+        }
+      ]
+    };
+
+    const instance = getInstance(
+      <Cascader
+        defaultOpen
+        data={itemsWithChildrenKey.data}
+        childrenKey={itemsWithChildrenKey.childrenKey}
+      />
+    );
+
+    const overlay = getDOMNode(instance.menuWrapperRef.current);
+
+    ReactTestUtils.act(() => {
+      const input = overlay.querySelector('.rs-picker-search-bar-input');
+
+      ReactTestUtils.Simulate.focus(input);
+
+      input.value = 'g';
+      ReactTestUtils.Simulate.change(input);
+    });
+
+    ReactTestUtils.act(() => {
+      const searchResult = overlay.querySelectorAll('.rs-picker-cascader-row');
+
+      assert.equal(searchResult.length, 2);
+    });
+  });
 });

package/src/Slider/Slider.d.ts

@@ -1,7 +1,9 @@
 import * as React from 'react';
-import { StandardProps } from '../@types/common';
+import { StandardProps, FormControlBaseProps } from '../@types/common';
 
-export interface SliderProps<ValueType = number> extends StandardProps {
+export interface SliderProps<ValueType = number>
+  extends StandardProps,
+    FormControlBaseProps<ValueType> {
   /** Minimum value of sliding range */
   min?: number;
 
@@ -11,12 +13,6 @@ export interface SliderProps<ValueType = number> extends StandardProps {
   /** Slide the value of one step */
   step?: number;
 
-  /** Value (Controlled) */
-  value?: ValueType;
-
-  /** Default value */
-  defaultValue?: ValueType;
-
   /** A css class to apply to the Handle node. */
   handleClassName?: string;
 
@@ -44,9 +40,6 @@ export interface SliderProps<ValueType = number> extends StandardProps {
   /** Vertical Slide */
   vertical?: boolean;
 
-  /** Callback function that changes data */
-  onChange?: (value: ValueType, event: React.MouseEvent) => void;
-
   /** Customize labels on the render ruler */
   renderMark?: (mark: number) => React.ReactNode;
 }