rsuite 5.53.0 → 5.53.2

package/CHANGELOG.md

@@ -1,3 +1,23 @@
+## [5.53.2](https://github.com/rsuite/rsuite/compare/v5.53.1...v5.53.2) (2024-02-06)
+
+
+### Bug Fixes
+
+* **CheckPicker:** remove debug script ([#3615](https://github.com/rsuite/rsuite/issues/3615)) ([62285da](https://github.com/rsuite/rsuite/commit/62285da3f238dceeb2f75c168cc07bbacf0a52ce))
+* **Tree,CheckTree:** fix `classPrefix` and `className` not working correctly ([#3616](https://github.com/rsuite/rsuite/issues/3616)) ([c82cb91](https://github.com/rsuite/rsuite/commit/c82cb919a12be2bd94e7bf1018e3f3c8aa522c1f))
+
+
+
+## [5.53.1](https://github.com/rsuite/rsuite/compare/v5.53.0...v5.53.1) (2024-02-06)
+
+
+### Bug Fixes
+
+* supports Fragment children component ([#3607](https://github.com/rsuite/rsuite/issues/3607)) ([299b951](https://github.com/rsuite/rsuite/commit/299b951644ce6564bd53f6d0a4d031180d9e4e9d))
+* **Panel:** fix uncontrolled re-rendering of Panel children ([#3610](https://github.com/rsuite/rsuite/issues/3610)) ([8f9a8a8](https://github.com/rsuite/rsuite/commit/8f9a8a831156ac1eeee50b8093bce9ed89cf043d))
+
+
+
 # [5.53.0](https://github.com/rsuite/rsuite/compare/v5.52.0...v5.53.0) (2024-02-02)
 
 ### Bug Fixes

package/cjs/Carousel/Carousel.js

@@ -97,7 +97,7 @@ var Carousel = /*#__PURE__*/_react.default.forwardRef(function (props, ref) {
   var uniqueId = (0, _react.useMemo)(function () {
     return (0, _utils.guid)();
   }, []);
-  var items = _react.default.Children.map(children, function (child, index) {
+  var items = _utils.ReactChildren.map(children, function (child, index) {
     var _extends2;
     if (!child) {
       return;

package/cjs/CheckTreePicker/CheckTreePicker.js

@@ -135,7 +135,7 @@ var CheckTreePicker = /*#__PURE__*/_react.default.forwardRef(function (props, re
   var _useClassNames = (0, _utils.useClassNames)(classPrefix),
     prefix = _useClassNames.prefix,
     merge = _useClassNames.merge;
-  var _useClassNames2 = (0, _utils.useClassNames)('check-tree'),
+  var _useClassNames2 = (0, _utils.useClassNames)(inline && classPrefix !== 'picker' ? classPrefix : 'check-tree'),
     checkTreePrefix = _useClassNames2.prefix,
     withCheckTreeClassPrefix = _useClassNames2.withClassPrefix;
   var _useControlled = (0, _utils.useControlled)(controlledValue, defaultValue),
@@ -607,8 +607,11 @@ var CheckTreePicker = /*#__PURE__*/_react.default.forwardRef(function (props, re
     }, nodeProps));
   };
   var renderCheckTree = function renderCheckTree() {
-    var _withCheckTreeClassPr, _merge2;
-    var classes = withCheckTreeClassPrefix((_withCheckTreeClassPr = {}, _withCheckTreeClassPr[className !== null && className !== void 0 ? className : ''] = inline, _withCheckTreeClassPr['without-children'] = !(0, _utils2.isSomeNodeHasChildren)(data, childrenKey), _withCheckTreeClassPr.virtualized = virtualized, _withCheckTreeClassPr));
+    var _merge2, _merge3;
+    var classes = merge(withCheckTreeClassPrefix({
+      'without-children': !(0, _utils2.isSomeNodeHasChildren)(data, childrenKey),
+      virtualized: virtualized
+    }), (_merge2 = {}, _merge2[className !== null && className !== void 0 ? className : ''] = inline, _merge2));
     var formattedNodes = getFormattedNodes(renderNode);
     if (!formattedNodes.some(function (v) {
       return v !== null;
@@ -617,18 +620,25 @@ var CheckTreePicker = /*#__PURE__*/_react.default.forwardRef(function (props, re
         className: prefix('none')
       }, locale.noResultsText);
     }
-    var treeNodesClass = merge(checkTreePrefix('root'), (_merge2 = {}, _merge2[checkTreePrefix('all-uncheckable')] = (0, _utils2.isEveryFirstLevelNodeUncheckable)(flattenNodes, uncheckableItemValues, valueKey), _merge2));
-    return /*#__PURE__*/_react.default.createElement(_Picker.TreeView, {
-      ref: inline ? root : treeView,
+    var treeNodesClass = merge(checkTreePrefix('root'), (_merge3 = {}, _merge3[checkTreePrefix('all-uncheckable')] = (0, _utils2.isEveryFirstLevelNodeUncheckable)(flattenNodes, uncheckableItemValues, valueKey), _merge3));
+    var treeViewProps = {
+      ref: treeView
+    };
+    if (inline) {
+      treeViewProps = (0, _extends2.default)({
+        ref: root,
+        style: (0, _extends2.default)({
+          height: height
+        }, style)
+      }, rest);
+    }
+    return /*#__PURE__*/_react.default.createElement(_Picker.TreeView, (0, _extends2.default)({}, treeViewProps, {
       multiselectable: true,
       treeRootClassName: treeNodesClass,
       className: classes,
-      style: inline ? (0, _extends2.default)({
-        height: height
-      }, style) : {},
       onScroll: onScroll,
       onKeyDown: inline ? handleTreeKeydown : undefined
-    }, virtualized ? /*#__PURE__*/_react.default.createElement(_Windowing.AutoSizer, {
+    }), virtualized ? /*#__PURE__*/_react.default.createElement(_Windowing.AutoSizer, {
       defaultHeight: inline ? height : menuMaxHeight,
       style: {
         width: 'auto',

package/cjs/DateRangePicker/DateRangePicker.js

@@ -628,7 +628,7 @@ var DateRangePicker = /*#__PURE__*/_react.default.forwardRef(function (props, re
       disabledShortcut: disabledShortcutButton,
       onShortcutClick: handleShortcutPageDate,
       "data-testid": "daterange-predefined-side"
-    }), /*#__PURE__*/_react.default.createElement(_react.default.Fragment, null, /*#__PURE__*/_react.default.createElement("div", {
+    }), /*#__PURE__*/_react.default.createElement(_Stack.default.Item, null, /*#__PURE__*/_react.default.createElement("div", {
       className: prefix('daterange-content')
     }, showHeader && /*#__PURE__*/_react.default.createElement("div", {
       className: prefix('daterange-header'),

package/cjs/Nav/Nav.js

@@ -68,8 +68,7 @@ var Nav = /*#__PURE__*/_react.default.forwardRef(function (props, ref) {
   }));
   var _ref = sidenav || {},
     activeKeyFromSidenav = _ref.activeKey,
-    _ref$onSelect = _ref.onSelect,
-    onSelectFromSidenav = _ref$onSelect === void 0 ? onSelectProp : _ref$onSelect;
+    onSelectFromSidenav = _ref.onSelect;
   var _useControlled = (0, _utils.useControlled)(activeKeyProp !== null && activeKeyProp !== void 0 ? activeKeyProp : activeKeyFromSidenav, defaultActiveKey),
     activeKey = _useControlled[0],
     setActiveKey = _useControlled[1];

package/cjs/Panel/Panel.js

@@ -81,14 +81,14 @@ var Panel = /*#__PURE__*/_react.default.forwardRef(function (props, ref) {
     onGroupSelect === null || onGroupSelect === void 0 ? void 0 : onGroupSelect(eventKey, event);
     setExpanded(!expanded);
   });
-  var renderBody = (0, _utils.useEventCallback)(function (bodyProps) {
+  var renderBody = function renderBody(bodyProps) {
     var classes = prefix('body', {
       'body-fill': bodyFill
     });
     return /*#__PURE__*/_react.default.createElement("div", (0, _extends2.default)({}, bodyProps, {
       className: classes
     }), children);
-  });
+  };
   var renderCollapsibleBody = function renderCollapsibleBody() {
     return /*#__PURE__*/_react.default.createElement(_Collapse.default, {
       in: expanded,

package/cjs/Stack/Stack.js

@@ -58,12 +58,12 @@ var Stack = /*#__PURE__*/_react.default.forwardRef(function (props, ref) {
    * toArray remove undefined, null and boolean
    */
   var filterChildren = _react.default.Children.toArray(children);
-  var count = filterChildren.length;
+  var count = _utils.ReactChildren.count(filterChildren);
   return /*#__PURE__*/_react.default.createElement(Component, (0, _extends2.default)({}, rest, {
     ref: ref,
     className: classes,
     style: styles
-  }), _react.default.Children.map(filterChildren, function (child, index) {
+  }), _utils.ReactChildren.map(filterChildren, function (child, index) {
     var childNode = childrenRenderMode === 'wrap' && child.type !== _StackItem.default ? /*#__PURE__*/_react.default.createElement(_StackItem.default, {
       key: index,
       className: prefix('item'),

package/cjs/Steps/Steps.js

@@ -40,7 +40,7 @@ var Steps = /*#__PURE__*/_react.default.forwardRef(function (props, ref) {
     vertical: vertical,
     horizontal: !vertical
   }));
-  var count = _react.default.Children.count(children);
+  var count = _utils.ReactChildren.count(children);
   var items = _utils.ReactChildren.mapCloneElement(children, function (item, index) {
     var itemStyles = {
       flexBasis: index < count - 1 ? 100 / (count - 1) + "%" : undefined,

package/cjs/Tabs/Tabs.d.ts

@@ -35,7 +35,7 @@ export interface TabsProps extends WithAsProps {
      * @param event
      * @returns
      */
-    onSelect?: (eventKey: string, event: React.MouseEvent) => void;
+    onSelect?: (eventKey: string | undefined, event: React.SyntheticEvent) => void;
 }
 interface TabsComponent extends RsRefForwardingComponent<'div', TabsProps> {
     Tab: typeof Tab;

package/cjs/Timeline/Timeline.js

@@ -33,7 +33,7 @@ var Timeline = /*#__PURE__*/_react.default.forwardRef(function (props, ref) {
   var _useClassNames = (0, _utils.useClassNames)(classPrefix),
     merge = _useClassNames.merge,
     withClassPrefix = _useClassNames.withClassPrefix;
-  var count = _react.default.Children.count(children);
+  var count = _utils.ReactChildren.count(children);
   var withTime = (0, _some.default)(_react.default.Children.toArray(children), function (item) {
     var _item$props;
     return item === null || item === void 0 ? void 0 : (_item$props = item.props) === null || _item$props === void 0 ? void 0 : _item$props.time;

package/cjs/TreePicker/TreePicker.js

@@ -162,7 +162,7 @@ var TreePicker = /*#__PURE__*/_react.default.forwardRef(function (props, ref) {
   var _useClassNames = (0, _utils.useClassNames)(classPrefix),
     prefix = _useClassNames.prefix,
     merge = _useClassNames.merge;
-  var _useClassNames2 = (0, _utils.useClassNames)('tree'),
+  var _useClassNames2 = (0, _utils.useClassNames)(inline && classPrefix !== 'picker' ? classPrefix : 'tree'),
     treePrefix = _useClassNames2.prefix,
     withTreeClassPrefix = _useClassNames2.withClassPrefix;
   var _useTreeSearch = (0, _treeUtils.useTreeSearch)({
@@ -589,18 +589,27 @@ var TreePicker = /*#__PURE__*/_react.default.forwardRef(function (props, ref) {
     }, nodeProps));
   };
   var renderTree = function renderTree() {
-    var _withTreeClassPrefix;
-    var classes = withTreeClassPrefix((_withTreeClassPrefix = {}, _withTreeClassPrefix[className !== null && className !== void 0 ? className : ''] = inline, _withTreeClassPrefix.virtualized = virtualized, _withTreeClassPrefix));
+    var _merge2;
+    var classes = merge(withTreeClassPrefix({
+      virtualized: virtualized
+    }), (_merge2 = {}, _merge2[className !== null && className !== void 0 ? className : ''] = inline, _merge2));
     var formattedNodes = getFormattedNodes(renderNode);
-    return /*#__PURE__*/_react.default.createElement(_Picker.TreeView, {
+    var treeViewProps = {
+      ref: treeView
+    };
+    if (inline) {
+      treeViewProps = (0, _extends2.default)({
+        ref: root,
+        style: (0, _extends2.default)({
+          height: height
+        }, style),
+        onKeyDown: handleTreeKeyDown
+      }, rest);
+    }
+    return /*#__PURE__*/_react.default.createElement(_Picker.TreeView, (0, _extends2.default)({}, treeViewProps, {
       treeRootClassName: treePrefix('root'),
-      ref: inline ? root : treeView,
-      className: classes,
-      style: inline ? (0, _extends2.default)({
-        height: height
-      }, style) : {},
-      onKeyDown: inline ? handleTreeKeyDown : undefined
-    }, virtualized ? /*#__PURE__*/_react.default.createElement(_Windowing.AutoSizer, {
+      className: classes
+    }), virtualized ? /*#__PURE__*/_react.default.createElement(_Windowing.AutoSizer, {
       defaultHeight: inline ? height : menuMaxHeight,
       style: {
         width: 'auto',

package/cjs/internals/Picker/ListCheckItem.js

@@ -70,7 +70,6 @@ var ListCheckItem = /*#__PURE__*/_react.default.forwardRef(function (props, ref)
     onClick: handleSelectItem,
     onCheckboxClick: handleCheck
   };
-  console.log(className, 'className');
   return /*#__PURE__*/_react.default.createElement(Component, (0, _extends2.default)({
     role: "option",
     "aria-selected": active,

package/cjs/utils/ReactChildren.d.ts

@@ -1,7 +1,7 @@
 import React from 'react';
 export declare function find(children: React.ReactNode, func: any, context?: any): React.ReactNode;
-export declare function map(children: React.ReactNode, func: any, context?: any): any;
-export declare function mapCloneElement(children: React.ReactNode, func: any, context?: any): any;
+export declare function map(children: React.ReactNode, func: any, context?: any): any[];
+export declare function mapCloneElement(children: React.ReactNode, func: any, context?: any): any[];
 export declare function count(children: React.ReactNode): number;
 declare function some(children: React.ReactNode, func: any, context?: any): boolean;
 declare const _default: {

package/cjs/utils/ReactChildren.js

@@ -18,14 +18,19 @@ function typeOf(object) {
 function isFragment(children) {
   return _react.default.Children.count(children) === 1 && typeOf(children) === Symbol.for('react.fragment');
 }
-function getChildren(children) {
-  var _props;
-  return isFragment(children) ? (_props = children.props) === null || _props === void 0 ? void 0 : _props.children : children;
+function flatChildren(children) {
+  return _react.default.Children.toArray(_react.default.Children.map(children, function (child) {
+    if (isFragment(child)) {
+      var _child$props;
+      return _react.default.Children.toArray(((_child$props = child.props) === null || _child$props === void 0 ? void 0 : _child$props.children) || []);
+    }
+    return child;
+  }));
 }
 function find(children, func, context) {
   var index = 0;
   var result;
-  _react.default.Children.forEach(getChildren(children), function (child) {
+  _react.default.Children.forEach(flatChildren(children), function (child) {
     if (result) {
       return;
     }
@@ -38,7 +43,7 @@ function find(children, func, context) {
 }
 function map(children, func, context) {
   var index = 0;
-  return _react.default.Children.map(getChildren(children), function (child) {
+  return _react.default.Children.map(flatChildren(children), function (child) {
     if (! /*#__PURE__*/_react.default.isValidElement(child)) {
       return child;
     }
@@ -55,14 +60,12 @@ function mapCloneElement(children, func, context) {
   }, context);
 }
 function count(children) {
-  return _react.default.Children.count(Array.isArray(children) ? children.filter(function (child) {
-    return child;
-  }) : children);
+  return _react.default.Children.count(flatChildren(children));
 }
 function some(children, func, context) {
   var index = 0;
   var result = false;
-  _react.default.Children.forEach(getChildren(children), function (child) {
+  _react.default.Children.forEach(flatChildren(children), function (child) {
     if (result) {
       return;
     }