@douyinfe/semi-ui 2.3.0-beta.0 → 2.4.0

package/lib/es/avatar/avatarGroup.js

@@ -115,8 +115,7 @@ export default class AvatarGroup extends PureComponent {
     }
 
     return /*#__PURE__*/React.createElement("div", {
-      className: groupCls,
-      "aria-label": "avatar group"
+      className: groupCls
     }, inner);
   }
 

package/lib/es/avatar/index.js

@@ -158,7 +158,6 @@ export default class Avatar extends BaseComponent {
     const hoverRender = hoverContent ? /*#__PURE__*/React.createElement("div", {
       className: "".concat(prefixCls, "-hover")
     }, hoverContent) : null;
-    let ariaLabel;
 
     if (isImg) {
       content = /*#__PURE__*/React.createElement("img", _Object$assign({
@@ -167,22 +166,18 @@ export default class Avatar extends BaseComponent {
         onError: this.handleError,
         alt: alt
       }, imgAttr));
-      ariaLabel = 'avatar';
     } else if (typeof children === 'string') {
       content = /*#__PURE__*/React.createElement("span", {
         className: "".concat(prefixCls, "-content")
       }, /*#__PURE__*/React.createElement("span", {
         className: "".concat(prefixCls, "-label")
       }, children));
-      ariaLabel = "avatar of ".concat(children);
     }
 
     return (
       /*#__PURE__*/
       // eslint-disable-next-line jsx-a11y/no-static-element-interactions,jsx-a11y/click-events-have-key-events
-      React.createElement("span", _Object$assign({
-        "aria-label": ariaLabel
-      }, others, {
+      React.createElement("span", _Object$assign({}, others, {
         style: style,
         className: avatarCls,
         onClick: onClick,

package/lib/es/collapse/index.js

@@ -99,8 +99,7 @@ class Collapse extends BaseComponent {
     } = this.state;
     return /*#__PURE__*/React.createElement("div", {
       className: clsPrefix,
-      style: style,
-      "aria-label": "Collapse"
+      style: style
     }, /*#__PURE__*/React.createElement(CollapseContext.Provider, {
       value: {
         activeSet,

package/lib/es/collapse/item.js

@@ -60,9 +60,7 @@ export default class CollapsePanel extends PureComponent {
     if (typeof header === 'string') {
       return /*#__PURE__*/React.createElement(React.Fragment, null, iconPosLeft ? icon : null, /*#__PURE__*/React.createElement("span", null, header), /*#__PURE__*/React.createElement("span", {
         className: "".concat(cssClasses.PREFIX, "-header-right")
-      }, /*#__PURE__*/React.createElement("span", {
-        "aria-label": 'Extra of collapse header'
-      }, extra), iconPosLeft ? null : icon));
+      }, /*#__PURE__*/React.createElement("span", null, extra), iconPosLeft ? null : icon));
     }
 
     return /*#__PURE__*/React.createElement(React.Fragment, null, iconPosLeft ? icon : null, header, iconPosLeft ? null : icon);
@@ -97,7 +95,6 @@ export default class CollapsePanel extends PureComponent {
       ["".concat(cssClasses.PREFIX, "-content")]: true
     });
     return /*#__PURE__*/React.createElement("div", _Object$assign({
-      "aria-label": 'Collapse panel',
       className: itemCls
     }, restProps), /*#__PURE__*/React.createElement("div", {
       role: "button",
@@ -113,7 +110,6 @@ export default class CollapsePanel extends PureComponent {
       reCalcKey: reCalcKey
     }, /*#__PURE__*/React.createElement("div", {
       className: contentCls,
-      "aria-label": 'Collapse content',
       "aria-hidden": !active,
       id: this.ariaID
     }, /*#__PURE__*/React.createElement("div", {

package/lib/es/dropdown/index.d.ts

@@ -12,11 +12,18 @@ export { DropdownDividerProps } from './dropdownDivider';
 export { DropdownItemProps, Type } from './dropdownItem';
 export { DropdownMenuProps } from './dropdownMenu';
 export { DropdownTitleProps } from './dropdownTitle';
-export interface DropDownMenuItemBasic {
-    node: 'title' | 'item' | 'divider';
+export interface DropDownMenuItemItem extends DropdownItemProps {
+    node: 'item';
     name?: string;
 }
-export declare type DropDownMenuItem = DropDownMenuItemBasic & DropdownItemProps & DropdownDividerProps & DropdownTitleProps;
+export interface DropDownMenuItemDivider extends DropdownDividerProps {
+    node: 'divider';
+}
+export interface DropDownMenuItemTitle extends DropdownTitleProps {
+    node: 'title';
+    name?: string;
+}
+export declare type DropDownMenuItem = DropDownMenuItemItem | DropDownMenuItemDivider | DropDownMenuItemTitle;
 export interface DropdownProps extends TooltipProps {
     render?: React.ReactNode;
     children?: React.ReactNode;

package/lib/es/empty/index.js

@@ -98,8 +98,7 @@ export default class Empty extends BaseComponent {
     };
     return /*#__PURE__*/React.createElement("div", {
       className: wrapperCls,
-      style: style,
-      "aria-label": "empty placeholder image"
+      style: style
     }, /*#__PURE__*/React.createElement("div", {
       className: "".concat(prefixCls, "-image"),
       style: imageStyle

package/lib/es/input/textarea.js

@@ -241,14 +241,16 @@ class TextArea extends BaseComponent {
       minLength: stateMinLength
     } = this.state;
     const wrapperCls = cls(className, "".concat(prefixCls, "-textarea-wrapper"), {
-      ["".concat(prefixCls, "-textarea-wrapper-disabled")]: disabled || readonly,
+      ["".concat(prefixCls, "-textarea-wrapper-disabled")]: disabled,
+      ["".concat(prefixCls, "-textarea-wrapper-readonly")]: readonly,
       [_concatInstanceProperty(_context = "".concat(prefixCls, "-textarea-wrapper-")).call(_context, validateStatus)]: Boolean(validateStatus),
       ["".concat(prefixCls, "-textarea-wrapper-focus")]: isFocus // [`${prefixCls}-textarea-wrapper-resize`]: !autosize && resize,
 
     }); // const ref = this.props.forwardRef || this.textAreaRef;
 
     const itemCls = cls("".concat(prefixCls, "-textarea"), {
-      ["".concat(prefixCls, "-textarea-disabled")]: disabled || readonly,
+      ["".concat(prefixCls, "-textarea-disabled")]: disabled,
+      ["".concat(prefixCls, "-textarea-readonly")]: readonly,
       ["".concat(prefixCls, "-textarea-autosize")]: autosize,
       ["".concat(prefixCls, "-textarea-showClear")]: showClear
     });

package/lib/es/inputNumber/index.js

@@ -412,9 +412,10 @@ class InputNumber extends BaseComponent {
       innerButtons,
       style,
       onNumberChange,
-      keepFocus
+      keepFocus,
+      defaultValue
     } = _a,
-          rest = __rest(_a, ["disabled", "className", "prefixCls", "min", "max", "step", "shiftStep", "precision", "formatter", "parser", "forwardedRef", "onUpClick", "onDownClick", "pressInterval", "pressTimeout", "suffix", "size", "hideButtons", "innerButtons", "style", "onNumberChange", "keepFocus"]);
+          rest = __rest(_a, ["disabled", "className", "prefixCls", "min", "max", "step", "shiftStep", "precision", "formatter", "parser", "forwardedRef", "onUpClick", "onDownClick", "pressInterval", "pressTimeout", "suffix", "size", "hideButtons", "innerButtons", "style", "onNumberChange", "keepFocus", "defaultValue"]);
 
     const {
       value,

package/lib/es/list/index.d.ts

@@ -5,7 +5,7 @@ import ListItem from './item';
 import { Grid } from './list-context';
 import BaseComponent from '../_base/baseComponent';
 export { ListItemProps } from './item';
-export interface ListProps {
+export interface ListProps<T> {
     style?: React.CSSProperties;
     className?: string;
     bordered?: boolean;
@@ -15,15 +15,15 @@ export interface ListProps {
     size?: 'small' | 'large' | 'default';
     split?: boolean;
     emptyContent?: React.ReactNode;
-    dataSource?: any[];
-    renderItem?: (item: any, ind: number) => React.ReactNode;
+    dataSource?: T[];
+    renderItem?: (item: T, ind: number) => React.ReactNode;
     grid?: Grid;
     loading?: boolean;
     loadMore?: React.ReactNode;
     onClick?: React.MouseEventHandler<HTMLLIElement>;
     onRightClick?: React.MouseEventHandler<HTMLLIElement>;
 }
-declare class List extends BaseComponent<ListProps> {
+declare class List<T = any> extends BaseComponent<ListProps<T>> {
     static Item: typeof ListItem;
     static propTypes: {
         style: PropTypes.Requireable<object>;

package/lib/es/navigation/Item.js

@@ -176,7 +176,7 @@ export default class NavItem extends BaseComponent {
     } else {
       let placeholderIcons = null;
 
-      if (mode === strings.MODE_VERTICAL && !limitIndent) {
+      if (mode === strings.MODE_VERTICAL && !limitIndent && !isCollapsed) {
         const iconAmount = icon && !indent ? level : level - 1;
         placeholderIcons = _times(iconAmount, () => this.renderIcon(null, strings.ICON_POS_RIGHT, false));
       }

package/lib/es/navigation/SubNav.js

@@ -200,7 +200,7 @@ export default class SubNav extends BaseComponent {
 
     let placeholderIcons = null;
 
-    if (mode === strings.MODE_VERTICAL && !limitIndent) {
+    if (mode === strings.MODE_VERTICAL && !limitIndent && !isCollapsed) {
       /* Different icons' amount means different indents.*/
       const iconAmount = icon && !indent ? level : level - 1;
       placeholderIcons = _times(iconAmount, index => this.renderIcon(null, strings.ICON_POS_RIGHT, false, undefined, index));

package/lib/es/scrollList/scrollItem.d.ts

@@ -3,7 +3,10 @@ import BaseComponent from '../_base/baseComponent';
 import PropTypes from 'prop-types';
 import { Item, ScrollItemAdapter } from '@douyinfe/semi-foundation/lib/es/scrollList/itemFoundation';
 import { Motion } from '../_base/base';
-declare type DebounceSelectFn = (e: React.UIEvent, newSelectedNode: HTMLElement) => void;
+interface DebounceSelectFn {
+    (e: React.UIEvent, newSelectedNode: HTMLElement): void;
+    cancel(): void;
+}
 export interface ScrollItemProps<T extends Item> {
     mode?: string;
     cycled?: boolean;
@@ -52,6 +55,7 @@ export default class ScrollItem<T extends Item> extends BaseComponent<ScrollItem
     debouncedSelect: DebounceSelectFn;
     constructor(props?: {});
     get adapter(): ScrollItemAdapter<ScrollItemProps<T>, ScrollItemState, T>;
+    componentWillUnmount(): void;
     componentDidMount(): void;
     componentDidUpdate(prevProps: ScrollItemProps<T>): void;
     _cacheNode: (name: string, node: Element) => Element;

package/lib/es/scrollList/scrollItem.js

@@ -429,6 +429,13 @@ export default class ScrollItem extends BaseComponent {
     });
   }
 
+  componentWillUnmount() {
+    if (this.props.cycled) {
+      this.throttledAdjustList.cancel();
+      this.debouncedSelect.cancel();
+    }
+  }
+
   componentDidMount() {
     this.foundation.init();
     const {

package/lib/es/select/index.js

@@ -506,7 +506,7 @@ class Select extends BaseComponent {
         role: "button",
         "aria-label": "Use the input box to create an optional item",
         onClick: e => this.onSelect(option, optionIndex, e),
-        key: new Date().valueOf()
+        key: option.key || option.label
       }, customCreateItem)
     );
   }

package/lib/es/sideSheet/SideSheetContent.js

@@ -79,8 +79,7 @@ export default class SideSheetContent extends React.PureComponent {
 
     if (title) {
       header = /*#__PURE__*/React.createElement("div", {
-        className: "".concat(prefixCls, "-title"),
-        "aria-label": 'Sidesheet title'
+        className: "".concat(prefixCls, "-title")
       }, this.props.title);
     }
 
@@ -99,6 +98,7 @@ export default class SideSheetContent extends React.PureComponent {
     return /*#__PURE__*/React.createElement("div", {
       className: "".concat(prefixCls, "-header"),
       role: 'heading',
+      "aria-level": 1,
       style: _Object$assign({}, headerStyle)
     }, header, closer);
   }
@@ -131,15 +131,12 @@ export default class SideSheetContent extends React.PureComponent {
       // onMouseDown={this.onDialogMouseDown}
       style: _Object$assign(_Object$assign({}, props.style), style)
     }, /*#__PURE__*/React.createElement("div", {
-      className: "".concat(prefixCls, "-content"),
-      "aria-label": "Sidesheet content"
+      className: "".concat(prefixCls, "-content")
     }, header, /*#__PURE__*/React.createElement("div", {
       className: "".concat(prefixCls, "-body"),
-      "aria-label": "Sidesheet body",
       style: props.bodyStyle
     }, props.children), props.footer ? /*#__PURE__*/React.createElement("div", {
-      className: "".concat(prefixCls, "-footer"),
-      "aria-label": "Sidesheet footer"
+      className: "".concat(prefixCls, "-footer")
     }, props.footer) : null));
     return dialogElement;
   }
@@ -161,8 +158,7 @@ export default class SideSheetContent extends React.PureComponent {
 
     return /*#__PURE__*/React.createElement("div", {
       className: wrapperCls,
-      style: wrapperStyle,
-      "aria-label": this.props['aria-label']
+      style: wrapperStyle
     }, this.getMaskElement(), this.getDialogElement());
   }
 

package/lib/es/spin/icon.js

@@ -44,7 +44,8 @@ function Icon() {
     viewBox: "0 0 36 36",
     version: "1.1",
     xmlns: "http://www.w3.org/2000/svg",
-    "aria-label": "spin"
+    "aria-hidden": true,
+    "data-icon": "spin"
   }), /*#__PURE__*/React.createElement("defs", null, /*#__PURE__*/React.createElement("linearGradient", {
     x1: "0%",
     y1: "100%",

package/lib/es/table/Table.js

@@ -1238,15 +1238,18 @@ class Table extends BaseComponent {
       const _dataSource = [...dataSource];
       const filteredSortedDataSource = this.foundation.getFilteredSortedDataSource(_dataSource, stateQueries);
       this.foundation.setCachedFilteredSortedDataSource(filteredSortedDataSource);
-      states.dataSource = filteredSortedDataSource; // when dataSource has change, should reset currentPage
-
-      states.pagination = _isObject(statePagination) ? _Object$assign(_Object$assign({}, statePagination), {
-        currentPage: _isObject(propsPagination) && propsPagination.currentPage ? propsPagination.currentPage : 1
-      }) : statePagination;
+      states.dataSource = filteredSortedDataSource;
 
       if (this.props.groupBy) {
         states.groups = null;
       }
+    } // when dataSource has change, should reset currentPage
+
+
+    if (dataSource !== prevProps.dataSource) {
+      states.pagination = _isObject(statePagination) ? _Object$assign(_Object$assign({}, statePagination), {
+        currentPage: _isObject(propsPagination) && propsPagination.currentPage ? propsPagination.currentPage : 1
+      }) : statePagination;
     }
 
     if (_Object$keys(states).length) {
@@ -1433,6 +1436,7 @@ class Table extends BaseComponent {
     return /*#__PURE__*/React.createElement("div", {
       ref: this.rootWrapRef,
       className: classnames(className, "".concat(prefixCls, "-wrapper")),
+      "data-column-fixed": anyColumnFixed,
       style: wrapStyle,
       id: id
     }, /*#__PURE__*/React.createElement(TableContextProvider, _Object$assign({}, tableContextValue), /*#__PURE__*/React.createElement(Spin, {

package/lib/es/timePicker/TimePicker.d.ts

@@ -69,6 +69,7 @@ export declare type TimePickerProps = {
     zIndex?: number | string;
     onBlur?: React.FocusEventHandler<HTMLInputElement>;
     onChange?: TimePickerAdapter['notifyChange'];
+    onChangeWithDateFirst?: boolean;
     onFocus?: React.FocusEventHandler<HTMLInputElement>;
     onOpenChange?: (open: boolean) => void;
 };
@@ -169,6 +170,7 @@ export default class TimePicker extends BaseComponent<TimePickerProps, TimePicke
         onFocus: (...args: any[]) => void;
         onBlur: (...args: any[]) => void;
         onChange: (...args: any[]) => void;
+        onChangeWithDateFirst: boolean;
         use12Hours: boolean;
         focusOnOpen: boolean;
         onKeyDown: (...args: any[]) => void;

package/lib/es/timePicker/TimePicker.js

@@ -149,9 +149,7 @@ export default class TimePicker extends BaseComponent {
       notifyOpenChange: function () {
         return _this2.props.onOpenChange(...arguments);
       },
-      notifyChange: function () {
-        return _this2.props.onChange && _this2.props.onChange(...arguments);
-      },
+      notifyChange: (agr1, arg2) => this.props.onChange && this.props.onChange(agr1, arg2),
       notifyFocus: function () {
         return _this2.props.onFocus && _this2.props.onFocus(...arguments);
       },
@@ -469,6 +467,7 @@ TimePicker.defaultProps = _Object$assign({
   onFocus: _noop,
   onBlur: _noop,
   onChange: _noop,
+  onChangeWithDateFirst: true,
   use12Hours: false,
   focusOnOpen: false,
   onKeyDown: _noop,

package/lib/es/timePicker/index.d.ts

@@ -91,6 +91,7 @@ export default class LocaleTimePicker extends React.PureComponent<LocalePickerPr
         onFocus: (...args: any[]) => void;
         onBlur: (...args: any[]) => void;
         onChange: (...args: any[]) => void;
+        onChangeWithDateFirst: boolean;
         use12Hours: boolean;
         focusOnOpen: boolean;
         onKeyDown: (...args: any[]) => void;

package/lib/es/tooltip/index.js

@@ -33,7 +33,7 @@ import Event from '@douyinfe/semi-foundation/lib/es/utils/Event';
 import { convertDOMRectToObject } from '@douyinfe/semi-foundation/lib/es/utils/dom';
 import TooltipFoundation from '@douyinfe/semi-foundation/lib/es/tooltip/foundation';
 import { strings, cssClasses, numbers } from '@douyinfe/semi-foundation/lib/es/tooltip/constants';
-import { getUuidv4 } from '@douyinfe/semi-foundation/lib/es/utils/uuid';
+import { getUuidShort } from '@douyinfe/semi-foundation/lib/es/utils/uuid';
 import '@douyinfe/semi-foundation/lib/es/tooltip/tooltip.css';
 import BaseComponent from '../_base/baseComponent';
 import { isHTMLElement } from '../_base/reactUtils';
@@ -268,7 +268,7 @@ export default class Tooltip extends BaseComponent {
       placement: props.position || 'top',
       transitionStyle: {},
       isPositionUpdated: false,
-      id: getUuidv4() // auto generate id, will be used by children.aria-descriptionby & content.id, improve a11y
+      id: getUuidShort() // auto generate id, will be used by children.aria-describedby & content.id, improve a11y
 
     };
     this.foundation = new TooltipFoundation(this.adapter);

package/lib/es/tree/treeNode.js

@@ -1,3 +1,4 @@
+import _isEmpty from "lodash/isEmpty";
 import _get from "lodash/get";
 import _isString from "lodash/isString";
 import _isFunction from "lodash/isFunction";
@@ -470,7 +471,14 @@ export default class TreeNode extends PureComponent {
           ref: this.setRef
         }, dragProps));
       } else {
-        return customLabel;
+        if (_isEmpty(style)) {
+          return customLabel;
+        } else {
+          // In virtualization, props.style will contain location information
+          return /*#__PURE__*/React.cloneElement(customLabel, {
+            style: _Object$assign(_Object$assign({}, _get(customLabel, ['props', 'style'])), style)
+          });
+        }
       }
     }
 

package/lib/es/treeSelect/index.js

@@ -17,7 +17,7 @@ import ReactDOM from 'react-dom';
 import cls from 'classnames';
 import PropTypes from 'prop-types';
 import TreeSelectFoundation from '@douyinfe/semi-foundation/lib/es/treeSelect/foundation';
-import { convertDataToEntities, flattenTreeData, calcExpandedKeysForValues, calcMotionKeys, findKeysForValues, calcCheckedKeys, calcExpandedKeys, getValueOrKey, normalizeKeyList, calcDisabledKeys, normalizeValue } from '@douyinfe/semi-foundation/lib/es/tree/treeUtil';
+import { convertDataToEntities, flattenTreeData, calcExpandedKeysForValues, calcMotionKeys, findKeysForValues, calcCheckedKeys, calcExpandedKeys, getValueOrKey, normalizeKeyList, calcDisabledKeys, normalizeValue, updateKeys } from '@douyinfe/semi-foundation/lib/es/tree/treeUtil';
 import { cssClasses, strings } from '@douyinfe/semi-foundation/lib/es/treeSelect/constants';
 import { numbers as popoverNumbers } from '@douyinfe/semi-foundation/lib/es/popover/constants';
 import { FixedSizeList as VirtualList } from 'react-window';
@@ -896,7 +896,7 @@ class TreeSelect extends BaseComponent {
     } // if treeData keys changes, we won't show animation
 
 
-    if (treeData && props.motion && !_isEqual(new _Set(_Object$keys(newState.keyEntities)), new _Set(_Object$keys(prevState.keyEntities)))) {
+    if (treeData && props.motion && !_isEqual(_Object$keys(newState.keyEntities), _Object$keys(prevState.keyEntities))) {
       if (prevProps && props.motion) {
         newState.motionKeys = new _Set([]);
         newState.motionType = null;
@@ -942,7 +942,11 @@ class TreeSelect extends BaseComponent {
         newState.selectedKeys = findKeysForValues(normalizeValue(props.defaultValue, withObject), valueEntities, isMultiple);
       } else if (treeData) {
         // If `treeData` changed, we also need check it
-        newState.selectedKeys = findKeysForValues(normalizeValue(props.value, withObject) || '', valueEntities, isMultiple);
+        if (props.value) {
+          newState.selectedKeys = findKeysForValues(normalizeValue(props.value, withObject) || '', valueEntities, isMultiple);
+        } else {
+          newState.selectedKeys = updateKeys(prevState.selectedKeys, keyEntities);
+        }
       }
     } else {
       // checkedKeys: multiple mode controlled || data changed
@@ -954,7 +958,11 @@ class TreeSelect extends BaseComponent {
         checkedKeyValues = findKeysForValues(normalizeValue(props.defaultValue, withObject), valueEntities, isMultiple);
       } else if (treeData) {
         // If `treeData` changed, we also need check it
-        checkedKeyValues = findKeysForValues(normalizeValue(props.value, withObject) || [], valueEntities, isMultiple);
+        if (props.value) {
+          checkedKeyValues = findKeysForValues(normalizeValue(props.value, withObject) || [], valueEntities, isMultiple);
+        } else {
+          checkedKeyValues = updateKeys(prevState.checkedKeys, keyEntities);
+        }
       }
 
       if (checkedKeyValues) {

package/lib/es/typography/util.js

@@ -62,7 +62,6 @@ const getRenderText = function (originEle, rows) {
   ellipsisContainer.style.zIndex = '-1000'; // clean up css overflow
 
   ellipsisContainer.style.textOverflow = 'clip';
-  ellipsisContainer.style.whiteSpace = 'normal';
   ellipsisContainer.style.webkitLineClamp = 'none'; // Render fake container
 
   ReactDOM.render( /*#__PURE__*/React.createElement(React.Fragment, null), ellipsisContainer); // Check if ellipsis in measure div is height enough for content

package/lib/es/upload/fileCard.js

@@ -223,12 +223,12 @@ class FileCard extends PureComponent {
       orbitStroke: '#FFF',
       "aria-label": "uploading file progress"
     }) : null, showRetry ? retry : null, showReplace && replace, showPicInfo && picInfo, !disabled && /*#__PURE__*/React.createElement("div", {
-      className: closeCls
+      className: closeCls,
+      onClick: e => this.onRemove(e)
     }, /*#__PURE__*/React.createElement(IconClose, {
       tabIndex: 0,
       role: "button",
-      size: "extra-small",
-      onClick: e => this.onRemove(e)
+      size: "extra-small"
     })), this.renderPicValidateMsg());
   }
 

package/list/index.tsx

@@ -14,7 +14,7 @@ import BaseComponent from '../_base/baseComponent';
 
 export { ListItemProps } from './item';
 
-export interface ListProps {
+export interface ListProps<T> {
     style?: React.CSSProperties;
     className?: string;
     bordered?: boolean;
@@ -24,8 +24,8 @@ export interface ListProps {
     size?: 'small' | 'large' | 'default';
     split?: boolean;
     emptyContent?: React.ReactNode;
-    dataSource?: any[];
-    renderItem?: (item: any, ind: number) => React.ReactNode;
+    dataSource?: T[];
+    renderItem?: (item: T, ind: number) => React.ReactNode;
     grid?: Grid;
     loading?: boolean;
     loadMore?: React.ReactNode;
@@ -35,7 +35,7 @@ export interface ListProps {
 
 const prefixCls = cssClasses.PREFIX;
 
-class List extends BaseComponent<ListProps> {
+class List<T = any> extends BaseComponent<ListProps<T>> {
     static Item = ListItem;
 
     static propTypes = {
@@ -170,4 +170,4 @@ class List extends BaseComponent<ListProps> {
     }
 }
 
-export default List;
+export default List;

package/navigation/Item.tsx

@@ -195,7 +195,7 @@ export default class NavItem extends BaseComponent<NavItemProps, NavItemState> {
             itemChildren = children;
         } else {
             let placeholderIcons = null;
-            if (mode === strings.MODE_VERTICAL && !limitIndent) {
+            if (mode === strings.MODE_VERTICAL && !limitIndent && !isCollapsed) {
                 const iconAmount = (icon && !indent) ? level : level - 1;
                 placeholderIcons = times(iconAmount, () => this.renderIcon(null, strings.ICON_POS_RIGHT, false));
             }

package/navigation/SubNav.tsx

@@ -237,7 +237,7 @@ export default class SubNav extends BaseComponent<SubNavProps, SubNavState> {
         }
 
         let placeholderIcons = null;
-        if (mode === strings.MODE_VERTICAL && !limitIndent) {
+        if (mode === strings.MODE_VERTICAL && !limitIndent && !isCollapsed) {
             /* Different icons' amount means different indents.*/
             const iconAmount = (icon && !indent) ? level : level - 1;
             placeholderIcons = times(iconAmount, index => this.renderIcon(null, strings.ICON_POS_RIGHT, false, undefined, index));

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@douyinfe/semi-ui",
-    "version": "2.3.0-beta.0",
+    "version": "2.4.0",
     "description": "",
     "main": "lib/cjs/index.js",
     "module": "lib/es/index.js",
@@ -14,11 +14,11 @@
     },
     "dependencies": {
         "@babel/runtime-corejs3": "^7.15.4",
-        "@douyinfe/semi-animation-react": "2.3.0-beta.0",
-        "@douyinfe/semi-foundation": "2.3.0-beta.0",
-        "@douyinfe/semi-icons": "2.3.0-beta.0",
-        "@douyinfe/semi-illustrations": "2.3.0-beta.0",
-        "@douyinfe/semi-theme-default": "2.3.0-beta.0",
+        "@douyinfe/semi-animation-react": "2.4.0",
+        "@douyinfe/semi-foundation": "2.4.0",
+        "@douyinfe/semi-icons": "2.4.0",
+        "@douyinfe/semi-illustrations": "2.4.0",
+        "@douyinfe/semi-theme-default": "2.4.0",
         "@types/react-window": "^1.8.2",
         "async-validator": "^3.5.0",
         "classnames": "^2.2.6",
@@ -68,13 +68,13 @@
     ],
     "author": "",
     "license": "MIT",
-    "gitHead": "5847f92fd90bc37f1fed5c7d2d6d64f6ebf632bc",
+    "gitHead": "d902a6f731475dea585d63907aee06ebe1153419",
     "devDependencies": {
         "@babel/plugin-proposal-decorators": "^7.15.8",
         "@babel/plugin-transform-runtime": "^7.15.8",
         "@babel/preset-env": "^7.15.8",
         "@babel/preset-react": "^7.14.5",
-        "@douyinfe/semi-scss-compile": "2.3.0-beta.0",
+        "@douyinfe/semi-scss-compile": "2.4.0",
         "@storybook/addon-knobs": "^6.3.1",
         "@types/lodash": "^4.14.176",
         "babel-loader": "^8.2.2",

package/scrollList/scrollItem.tsx

@@ -13,8 +13,10 @@ import { Motion } from '../_base/base';
 const msPerFrame = 1000 / 60;
 const blankReg = /^\s*$/;
 const wheelMode = 'wheel';
-
-type DebounceSelectFn = (e: React.UIEvent, newSelectedNode: HTMLElement) => void;
+interface DebounceSelectFn {
+    (e: React.UIEvent, newSelectedNode: HTMLElement): void;
+    cancel(): void
+}
 export interface ScrollItemProps<T extends Item> {
     mode?: string;
     cycled?: boolean;
@@ -110,7 +112,12 @@ export default class ScrollItem<T extends Item> extends BaseComponent<ScrollItem
             scrollToCenter: this.scrollToCenter,
         };
     }
-
+    componentWillUnmount(){
+        if (this.props.cycled) {
+            this.throttledAdjustList.cancel();
+            this.debouncedSelect.cancel();
+        }
+    }
     componentDidMount() {
         this.foundation.init();
 

package/select/index.tsx

@@ -739,7 +739,12 @@ class Select extends BaseComponent<SelectProps, SelectState> {
 
         return (
             // eslint-disable-next-line jsx-a11y/click-events-have-key-events,jsx-a11y/interactive-supports-focus
-            <div role="button" aria-label="Use the input box to create an optional item" onClick={e => this.onSelect(option, optionIndex, e)} key={new Date().valueOf()}>
+            <div
+                role="button"
+                aria-label="Use the input box to create an optional item"
+                onClick={e => this.onSelect(option, optionIndex, e)}
+                key={option.key || option.label}
+            >
                 {customCreateItem}
             </div>
         );