@douyinfe/semi-ui 2.2.1 → 2.3.1

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

@@ -34,6 +34,11 @@ export declare type OverrideCommonProps = 'renderFullLabel' | 'renderLabel' | 'd
 * TreeSelectProps inherits some properties from BasicTreeSelectProps (from foundation) and TreeProps (from semi-ui-react).
 */
 export interface TreeSelectProps extends Omit<BasicTreeSelectProps, OverrideCommonProps | 'validateStatus' | 'searchRender'>, Pick<TreeProps, OverrideCommonProps> {
+    'aria-describedby'?: React.AriaAttributes['aria-describedby'];
+    'aria-errormessage'?: React.AriaAttributes['aria-errormessage'];
+    'aria-invalid'?: React.AriaAttributes['aria-invalid'];
+    'aria-labelledby'?: React.AriaAttributes['aria-labelledby'];
+    'aria-required'?: React.AriaAttributes['aria-required'];
     motion?: Motion;
     mouseEnterDelay?: number;
     mouseLeaveDelay?: number;
@@ -45,6 +50,7 @@ export interface TreeSelectProps extends Omit<BasicTreeSelectProps, OverrideComm
     dropdownMatchSelectWidth?: boolean;
     dropdownStyle?: React.CSSProperties;
     insetLabel?: React.ReactNode;
+    insetLabelId?: string;
     maxTagCount?: number;
     motionExpand?: boolean;
     optionListStyle?: React.CSSProperties;
@@ -85,6 +91,11 @@ export interface TreeSelectState extends Omit<BasicTreeSelectInnerData, Override
 declare class TreeSelect extends BaseComponent<TreeSelectProps, TreeSelectState> {
     static contextType: React.Context<import("../configProvider/context").ContextValue>;
     static propTypes: {
+        'aria-describedby': PropTypes.Requireable<string>;
+        'aria-errormessage': PropTypes.Requireable<string>;
+        'aria-invalid': PropTypes.Requireable<boolean>;
+        'aria-labelledby': PropTypes.Requireable<string>;
+        'aria-required': PropTypes.Requireable<boolean>;
         loadedKeys: PropTypes.Requireable<string[]>;
         loadData: PropTypes.Requireable<(...args: any[]) => any>;
         onLoad: PropTypes.Requireable<(...args: any[]) => any>;
@@ -136,6 +147,7 @@ declare class TreeSelect extends BaseComponent<TreeSelectProps, TreeSelectState>
         suffix: PropTypes.Requireable<PropTypes.ReactNodeLike>;
         prefix: PropTypes.Requireable<PropTypes.ReactNodeLike>;
         insetLabel: PropTypes.Requireable<PropTypes.ReactNodeLike>;
+        insetLabelId: PropTypes.Requireable<string>;
         zIndex: PropTypes.Requireable<number>;
         getPopupContainer: PropTypes.Requireable<(...args: any[]) => any>;
         dropdownMatchSelectWidth: PropTypes.Requireable<boolean>;
@@ -156,6 +168,7 @@ declare class TreeSelect extends BaseComponent<TreeSelectProps, TreeSelectState>
         optionListStyle: PropTypes.Requireable<object>;
         searchRender: PropTypes.Requireable<boolean | ((...args: any[]) => any)>;
         renderSelectedItem: PropTypes.Requireable<(...args: any[]) => any>;
+        'aria-label': PropTypes.Requireable<string>;
     };
     static defaultProps: Partial<TreeSelectProps>;
     inputRef: React.RefObject<typeof Input>;
@@ -167,6 +180,7 @@ declare class TreeSelect extends BaseComponent<TreeSelectProps, TreeSelectState>
     onNodeClick: any;
     onNodeDoubleClick: any;
     onMotionEnd: any;
+    treeSelectID: string;
     constructor(props: TreeSelectProps);
     static getDerivedStateFromProps(props: TreeSelectProps, prevState: TreeSelectState): Partial<TreeSelectState>;
     get adapter(): TreeSelectAdapter<TreeSelectProps, TreeSelectState>;
@@ -177,6 +191,7 @@ declare class TreeSelect extends BaseComponent<TreeSelectProps, TreeSelectState>
     renderContent: () => JSX.Element;
     removeTag: (removedKey: TreeNodeData['key']) => void;
     handleClick: (e: React.MouseEvent) => void;
+    handleSelectionEnterPress: (e: React.KeyboardEvent<HTMLDivElement>) => void;
     showClearBtn: () => boolean;
     renderTagList: () => React.ReactNode[];
     /**
@@ -189,6 +204,7 @@ declare class TreeSelect extends BaseComponent<TreeSelectProps, TreeSelectState>
     renderSingleTriggerSearch: () => JSX.Element;
     renderSelectContent: () => JSX.Element;
     handleClear: (e: React.MouseEvent) => void;
+    handleClearEnterPress: (e: React.KeyboardEvent<HTMLDivElement>) => void;
     handleMouseOver: (e: React.MouseEvent) => void;
     handleMouseLeave: (e: React.MouseEvent) => void;
     search: (value: string) => void;

package/lib/es/treeSelect/index.js

@@ -10,6 +10,7 @@ import _mapInstanceProperty from "@babel/runtime-corejs3/core-js-stable/instance
 import _concatInstanceProperty from "@babel/runtime-corejs3/core-js-stable/instance/concat";
 import _Promise from "@babel/runtime-corejs3/core-js-stable/promise";
 import _Set from "@babel/runtime-corejs3/core-js-stable/set";
+import _sliceInstanceProperty from "@babel/runtime-corejs3/core-js-stable/instance/slice";
 import _Object$keys from "@babel/runtime-corejs3/core-js-stable/object/keys";
 import React, { Fragment } from 'react';
 import ReactDOM from 'react-dom';
@@ -44,6 +45,8 @@ const key = 0;
 
 class TreeSelect extends BaseComponent {
   constructor(props) {
+    var _context2;
+
     super(props);
 
     this.renderSuffix = () => {
@@ -63,7 +66,8 @@ class TreeSelect extends BaseComponent {
     this.renderPrefix = () => {
       const {
         prefix,
-        insetLabel
+        insetLabel,
+        insetLabelId
       } = this.props;
       const labelNode = prefix || insetLabel;
       const prefixWrapperCls = cls({
@@ -74,7 +78,8 @@ class TreeSelect extends BaseComponent {
         ["".concat(prefixcls, "-prefix-icon")]: isSemiIcon(labelNode)
       });
       return /*#__PURE__*/React.createElement("div", {
-        className: prefixWrapperCls
+        className: prefixWrapperCls,
+        id: insetLabelId
       }, labelNode);
     };
 
@@ -94,7 +99,6 @@ class TreeSelect extends BaseComponent {
       const popoverCls = cls(dropdownClassName, "".concat(prefixcls, "-popover"));
       return /*#__PURE__*/React.createElement("div", {
         className: popoverCls,
-        role: "list-box",
         style: style
       }, this.renderTree());
     };
@@ -107,6 +111,10 @@ class TreeSelect extends BaseComponent {
       this.foundation.handleClick(e);
     };
 
+    this.handleSelectionEnterPress = e => {
+      this.foundation.handleSelectionEnterPress(e);
+    };
+
     this.showClearBtn = () => {
       const {
         searchPosition
@@ -275,6 +283,11 @@ class TreeSelect extends BaseComponent {
       this.foundation.handleClear(e);
     };
 
+    this.handleClearEnterPress = e => {
+      e && e.stopPropagation();
+      this.foundation.handleClearEnterPress(e);
+    };
+
     this.handleMouseOver = e => {
       this.foundation.toggleHoverState(true);
     };
@@ -312,8 +325,12 @@ class TreeSelect extends BaseComponent {
 
       if (showClearBtn) {
         return /*#__PURE__*/React.createElement("div", {
+          role: 'button',
+          tabIndex: 0,
+          "aria-label": "Clear TreeSelect value",
           className: clearCls,
-          onClick: this.handleClear
+          onClick: this.handleClear,
+          onKeyPress: this.handleClearEnterPress
         }, /*#__PURE__*/React.createElement(IconClear, null));
       }
 
@@ -397,11 +414,29 @@ class TreeSelect extends BaseComponent {
       }, showClear || isTriggerPositionSearch && inputValue ? this.renderClearBtn() : null), /*#__PURE__*/React.createElement(Fragment, {
         key: 'arrow'
       }, this.renderArrow())];
+      const tabIndex = disabled ? null : 0;
+      /**
+       * Reasons for disabling the a11y eslint rule:
+       * The following attributes(aria-controls,aria-expanded) will be automatically added by Tooltip, no need to declare here
+       */
+
       return /*#__PURE__*/React.createElement("div", _Object$assign({
+        // eslint-disable-next-line jsx-a11y/role-has-required-aria-props
+        role: 'combobox',
+        "aria-disabled": disabled,
+        "aria-haspopup": "tree",
+        tabIndex: tabIndex,
         className: classNames,
         style: style,
         ref: this.triggerRef,
-        onClick: this.handleClick
+        onClick: this.handleClick,
+        onKeyPress: this.handleSelectionEnterPress,
+        "aria-invalid": this.props['aria-invalid'],
+        "aria-errormessage": this.props['aria-errormessage'],
+        "aria-label": this.props['aria-label'],
+        "aria-labelledby": this.props['aria-labelledby'],
+        "aria-describedby": this.props['aria-describedby'],
+        "aria-required": this.props['aria-required']
       }, mouseEvent), inner);
     }; // eslint-disable-next-line @typescript-eslint/no-shadow
 
@@ -478,7 +513,8 @@ class TreeSelect extends BaseComponent {
         disabled,
         size,
         searchAutoFocus,
-        placeholder
+        placeholder,
+        maxTagCount
       } = this.props;
       const {
         keyEntities,
@@ -487,6 +523,7 @@ class TreeSelect extends BaseComponent {
       } = this.state;
       const keyList = normalizeKeyList(checkedKeys, keyEntities, leafOnly);
       return /*#__PURE__*/React.createElement(TagInput, {
+        maxTagCount: maxTagCount,
         disabled: disabled,
         onInputChange: v => this.search(v),
         ref: this.tagInputRef,
@@ -558,6 +595,7 @@ class TreeSelect extends BaseComponent {
         }
 
         return /*#__PURE__*/React.createElement(Input, _Object$assign({
+          "aria-label": 'Filter TreeSelect item',
           ref: this.inputRef,
           autofocus: searchAutoFocus,
           placeholder: placeholder
@@ -786,11 +824,11 @@ class TreeSelect extends BaseComponent {
           labelEllipsis: typeof labelEllipsis === 'undefined' ? virtualize : labelEllipsis
         }
       }, /*#__PURE__*/React.createElement("div", {
-        className: wrapperCls,
-        role: "list-box"
+        className: wrapperCls
       }, outerTopSlot, !outerTopSlot && filterTreeNode && isDropdownPositionSearch && this.renderInput(), /*#__PURE__*/React.createElement("div", {
         className: listCls,
         role: "tree",
+        "aria-multiselectable": multiple ? true : false,
         style: optionListStyle
       }, noData ? this.renderEmpty() : this.renderNodeList()), outerBottomSlot));
     };
@@ -827,6 +865,7 @@ class TreeSelect extends BaseComponent {
     this.optionsRef = /*#__PURE__*/React.createRef();
     this.clickOutsideHandler = null;
     this.foundation = new TreeSelectFoundation(this.adapter);
+    this.treeSelectID = _sliceInstanceProperty(_context2 = Math.random().toString(36)).call(_context2, 2);
   } // eslint-disable-next-line max-lines-per-function
 
 
@@ -1127,6 +1166,11 @@ class TreeSelect extends BaseComponent {
 
 TreeSelect.contextType = ConfigContext;
 TreeSelect.propTypes = {
+  'aria-describedby': PropTypes.string,
+  'aria-errormessage': PropTypes.string,
+  'aria-invalid': PropTypes.bool,
+  'aria-labelledby': PropTypes.string,
+  'aria-required': PropTypes.bool,
   loadedKeys: PropTypes.arrayOf(PropTypes.string),
   loadData: PropTypes.func,
   onLoad: PropTypes.func,
@@ -1179,6 +1223,7 @@ TreeSelect.propTypes = {
   suffix: PropTypes.node,
   prefix: PropTypes.node,
   insetLabel: PropTypes.node,
+  insetLabelId: PropTypes.string,
   zIndex: PropTypes.number,
   getPopupContainer: PropTypes.func,
   dropdownMatchSelectWidth: PropTypes.bool,
@@ -1198,7 +1243,8 @@ TreeSelect.propTypes = {
   labelEllipsis: PropTypes.bool,
   optionListStyle: PropTypes.object,
   searchRender: PropTypes.oneOfType([PropTypes.func, PropTypes.bool]),
-  renderSelectedItem: PropTypes.func
+  renderSelectedItem: PropTypes.func,
+  'aria-label': PropTypes.string
 };
 TreeSelect.defaultProps = {
   searchPosition: strings.SEARCH_POSITION_DROPDOWN,
@@ -1225,6 +1271,7 @@ TreeSelect.defaultProps = {
   onVisibleChange: _noop,
   expandAction: false,
   clickToHide: true,
-  searchAutoFocus: false
+  searchAutoFocus: false,
+  'aria-label': 'TreeSelect'
 };
 export default TreeSelect;

package/lib/es/typography/base.js

@@ -32,6 +32,7 @@ import Tooltip from '../tooltip/index';
 import Popover from '../popover/index';
 import getRenderText from './util';
 import warning from '@douyinfe/semi-foundation/lib/es/utils/warning';
+import isEnterPress from '@douyinfe/semi-foundation/lib/es/utils/isEnterPress';
 import LocaleConsumer from '../locale/localeConsumer';
 import { isSemiIcon } from '../_utils';
 const prefixCls = cssClasses.PREFIX;
@@ -234,13 +235,22 @@ export default class Base extends Component {
       }
 
       if (!noExpandText || !noCollapseText) {
-        return /*#__PURE__*/React.createElement("a", {
-          className: "".concat(prefixCls, "-ellipsis-expand"),
-          key: "expand",
-          ref: this.expandRef,
-          "aria-label": text,
-          onClick: this.toggleOverflow
-        }, text);
+        return (
+          /*#__PURE__*/
+          // TODO: replace `a` tag with `span` in next major version
+          // NOTE: may have effect on style
+          // eslint-disable-next-line jsx-a11y/anchor-is-valid
+          React.createElement("a", {
+            role: "button",
+            tabIndex: 0,
+            className: "".concat(prefixCls, "-ellipsis-expand"),
+            key: "expand",
+            ref: this.expandRef,
+            "aria-label": text,
+            onClick: this.toggleOverflow,
+            onKeyPress: e => isEnterPress(e) && this.toggleOverflow(e)
+          }, text)
+        );
       }
 
       return null;

package/lib/es/typography/copyable.js

@@ -9,6 +9,7 @@ import cls from 'classnames';
 import { noop } from '@douyinfe/semi-foundation/lib/es/utils/function';
 import LocaleConsumer from '../locale/localeConsumer';
 import { IconCopy, IconTick } from '@douyinfe/semi-icons';
+import isEnterPress from '@douyinfe/semi-foundation/lib/es/utils/isEnterPress';
 const prefixCls = cssClasses.PREFIX;
 export class Copyable extends React.PureComponent {
   constructor(props) {
@@ -100,7 +101,10 @@ export class Copyable extends React.PureComponent {
     }, /*#__PURE__*/React.createElement("a", {
       className: "".concat(prefixCls, "-action-copy-icon")
     }, /*#__PURE__*/React.createElement(IconCopy, {
-      onClick: this.copy
+      role: "button",
+      tabIndex: 0,
+      onClick: this.copy,
+      onKeyPress: e => isEnterPress(e) && this.copy(e)
     })))));
   }
 

package/lib/es/upload/fileCard.js

@@ -16,6 +16,8 @@ const prefixCls = cssClasses.PREFIX;
 const ErrorSvg = function () {
   let props = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   return /*#__PURE__*/React.createElement("svg", _Object$assign({
+    focusable: false,
+    "aria-hidden": true,
     width: "16",
     height: "16",
     viewBox: "0 0 16 16",
@@ -37,6 +39,8 @@ const ErrorSvg = function () {
 const ReplaceSvg = function () {
   let props = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   return /*#__PURE__*/React.createElement("svg", _Object$assign({
+    focusable: false,
+    "aria-hidden": true,
     width: "28",
     height: "28",
     viewBox: "0 0 28 28",
@@ -64,6 +68,8 @@ const ReplaceSvg = function () {
 const DirectorySvg = function () {
   let props = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   return /*#__PURE__*/React.createElement("svg", _Object$assign({
+    focusable: false,
+    "aria-hidden": true,
     width: "24",
     height: "24",
     viewBox: "0 0 24 24",
@@ -177,6 +183,8 @@ class FileCard extends PureComponent {
     });
     const closeCls = "".concat(prefixCls, "-picture-file-card-close");
     const retry = /*#__PURE__*/React.createElement("div", {
+      role: "button",
+      tabIndex: 0,
       className: "".concat(prefixCls, "-picture-file-card-retry"),
       onClick: e => this.onRetry(e)
     }, /*#__PURE__*/React.createElement(IconRefresh, {
@@ -189,6 +197,8 @@ class FileCard extends PureComponent {
       showArrow: false,
       spacing: 4
     }, /*#__PURE__*/React.createElement("div", {
+      role: "button",
+      tabIndex: 0,
       className: "".concat(prefixCls, "-picture-file-card-replace"),
       onClick: e => this.onReplace(e)
     }, /*#__PURE__*/React.createElement(ReplaceSvg, {
@@ -199,9 +209,10 @@ class FileCard extends PureComponent {
     }, index + 1);
     const thumbnail = typeof renderThumbnail === 'function' ? renderThumbnail(this.props) : /*#__PURE__*/React.createElement("img", {
       src: url,
-      alt: "picture of ".concat(name)
+      alt: name
     });
     return /*#__PURE__*/React.createElement("div", {
+      role: "listitem",
       className: filePicCardCls,
       style: style,
       onClick: onPreviewClick
@@ -209,12 +220,15 @@ class FileCard extends PureComponent {
       percent: percent,
       type: "circle",
       size: "small",
-      orbitStroke: '#FFF'
+      orbitStroke: '#FFF',
+      "aria-label": "uploading file progress"
     }) : null, showRetry ? retry : null, showReplace && replace, showPicInfo && picInfo, !disabled && /*#__PURE__*/React.createElement("div", {
-      className: closeCls
-    }, /*#__PURE__*/React.createElement(IconClose, {
-      size: "extra-small",
+      className: closeCls,
       onClick: e => this.onRemove(e)
+    }, /*#__PURE__*/React.createElement(IconClose, {
+      tabIndex: 0,
+      role: "button",
+      size: "extra-small"
     })), this.renderPicValidateMsg());
   }
 
@@ -250,7 +264,8 @@ class FileCard extends PureComponent {
     const showReplace = status === strings.FILE_STATUS_SUCCESS && propsShowReplace;
     const fileSize = this.transSize(size);
     let previewContent = preview ? /*#__PURE__*/React.createElement("img", {
-      src: url
+      src: url,
+      alt: name
     }) : /*#__PURE__*/React.createElement(IconFile, {
       size: "large"
     });
@@ -260,6 +275,7 @@ class FileCard extends PureComponent {
     }
 
     return /*#__PURE__*/React.createElement("div", {
+      role: "listitem",
       className: fileCardCls,
       style: style,
       onClick: onPreviewClick
@@ -289,12 +305,15 @@ class FileCard extends PureComponent {
       percent: percent,
       style: {
         width: '100%'
-      }
+      },
+      "aria-label": "uploading file progress"
     }) : null, /*#__PURE__*/React.createElement("div", {
       className: "".concat(infoCls, "-main-control")
     }, /*#__PURE__*/React.createElement("span", {
       className: "".concat(infoCls, "-validate-message")
     }, this.renderValidateMessage()), showRetry ? /*#__PURE__*/React.createElement("span", {
+      role: "button",
+      tabIndex: 0,
       className: "".concat(infoCls, "-retry"),
       onClick: e => this.onRetry(e)
     }, locale.retry) : null)), /*#__PURE__*/React.createElement(IconButton, {

package/lib/es/upload/index.js

@@ -208,6 +208,7 @@ class Upload extends BaseComponent {
       });
       const mainCls = "".concat(prefixCls, "-file-list-main");
       const addContentProps = {
+        role: 'button',
         className: uploadAddCls,
         onClick: this.onClick
       };
@@ -240,7 +241,9 @@ class Upload extends BaseComponent {
       return /*#__PURE__*/React.createElement(LocaleConsumer, {
         componentName: "Upload"
       }, locale => /*#__PURE__*/React.createElement("div", _Object$assign({}, containerProps), /*#__PURE__*/React.createElement("div", {
-        className: mainCls
+        className: mainCls,
+        role: "list",
+        "aria-label": "picture list"
       }, _mapInstanceProperty(fileList).call(fileList, (file, index) => this.renderFile(file, index, locale)), showAddTriggerInList ? addContent : null)));
     };
 
@@ -274,10 +277,14 @@ class Upload extends BaseComponent {
       }, /*#__PURE__*/React.createElement("span", {
         className: "".concat(titleCls, "-choosen")
       }, locale.selectedFiles), showClear ? /*#__PURE__*/React.createElement("span", {
+        role: "button",
+        tabIndex: 0,
         onClick: this.clear,
         className: "".concat(titleCls, "-clear")
       }, locale.clear) : null) : null, /*#__PURE__*/React.createElement("div", {
-        className: mainCls
+        className: mainCls,
+        role: "list",
+        "aria-label": "file list"
       }, _mapInstanceProperty(fileList).call(fileList, (file, index) => this.renderFile(file, index, locale)))));
     };
 
@@ -302,7 +309,8 @@ class Upload extends BaseComponent {
       const {
         draggable,
         children,
-        listType
+        listType,
+        disabled
       } = this.props;
       const uploadAddCls = cls("".concat(prefixCls, "-add"));
 
@@ -315,6 +323,9 @@ class Upload extends BaseComponent {
       }
 
       return /*#__PURE__*/React.createElement("div", {
+        role: "button",
+        tabIndex: 0,
+        "aria-disabled": disabled,
         className: uploadAddCls,
         onClick: this.onClick
       }, children);
@@ -328,7 +339,8 @@ class Upload extends BaseComponent {
         children,
         dragIcon,
         dragMainText,
-        dragSubText
+        dragSubText,
+        disabled
       } = this.props;
       const dragAreaBaseCls = "".concat(prefixCls, "-drag-area");
       const dragAreaCls = cls(dragAreaBaseCls, {
@@ -339,6 +351,9 @@ class Upload extends BaseComponent {
       return /*#__PURE__*/React.createElement(LocaleConsumer, {
         componentName: "Upload"
       }, locale => /*#__PURE__*/React.createElement("div", {
+        role: "button",
+        tabIndex: 0,
+        "aria-disabled": disabled,
         className: dragAreaCls,
         onDrop: this.onDrop,
         onDragOver: this.onDragOver,

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/list/item.tsx

@@ -86,7 +86,6 @@ export default class ListItem extends PureComponent<ListItemProps> {
         }
         let content = (
             <li
-                role="list-item"
                 className={itemCls}
                 style={style}
                 onClick={handleClick}

package/modal/Modal.tsx

@@ -283,6 +283,7 @@ class Modal extends BaseComponent<ModalReactProps, ModalState> {
             } else {
                 return (
                     <Button
+                        aria-label="cancel"
                         onClick={this.handleCancel}
                         loading={cancelLoading}
                         type="tertiary"
@@ -300,6 +301,7 @@ class Modal extends BaseComponent<ModalReactProps, ModalState> {
                     <div>
                         {getCancelButton(locale)}
                         <Button
+                            aria-label="confirm"
                             type={okType}
                             theme="solid"
                             loading={confirmLoading}