@jetbrains/ring-ui 6.0.41 → 6.0.43

package/components/date-picker/date-input.d.ts

@@ -43,6 +43,6 @@ export default class DateInput extends React.PureComponent<DateInputProps> {
     inputRef: (el: HTMLInputElement | null) => void;
     updateInput({ text, active }: UpdateInputConfig): void;
     handleChange: (e: React.ChangeEvent<HTMLInputElement>) => void;
-    handleKeyDown: (e: React.KeyboardEvent) => false | void;
+    handleKeyDown: (e: React.KeyboardEvent) => void;
     render(): React.JSX.Element;
 }

package/components/date-picker/date-input.js

@@ -56,7 +56,16 @@ export default class DateInput extends React.PureComponent {
         }
     }
     handleChange = (e) => this.props.onInput(e.currentTarget.value, e.currentTarget.dataset.name);
-    handleKeyDown = (e) => e.key === 'Enter' && this.props.onConfirm();
+    handleKeyDown = (e) => {
+        if (e.key === 'Enter') {
+            // We have to prevent the default behavior, because restoring focus by TabTrap caused by
+            // pressing Enter will trigger the onClick event on the DatePicker Dropdown anchor,
+            // so DatePicker will be open again.
+            // https://youtrack.jetbrains.com/issue/RG-2450/Anchor-should-be-focused-after-closing-datepicker#focus=Comments-27-10044234.0-0.
+            e.preventDefault();
+            this.props.onConfirm();
+        }
+    };
     render() {
         const { active, divider, text, time, name, hoverDate, date, displayFormat, translations, onActivate, onClear, fromPlaceholder, toPlaceholder, timePlaceholder, locale } = this.props;
         const { translate } = this.context;

package/components/select/select.js

@@ -799,9 +799,13 @@ export default class Select extends Component {
             [styles[`size${this.props.size}`]]: this.props.type !== Type.INLINE,
             [styles.disabled]: this.props.disabled
         });
-        const icons = this._getIcons();
-        const style = getStyle(icons.length);
-        const iconsNode = <div className={styles.icons}>{icons}</div>;
+        let style;
+        let iconsNode;
+        if (this.props.type === Type.INPUT || this.props.type === Type.BUTTON) {
+            const icons = this._getIcons();
+            style = getStyle(icons.length);
+            iconsNode = <div className={styles.icons}>{icons}</div>;
+        }
         const ariaProps = this.state.showPopup
             ? {
                 'aria-owns': this.listId,
@@ -823,7 +827,7 @@ export default class Select extends Component {
                         ...this.getShortcutsMap(),
                         ...this._popup?.list?.shortcutsMap
                     })
-                    : undefined} icon={this.props.filterIcon} afterInput={this.props.type === Type.INPUT && iconsNode}/>
+                    : undefined} icon={this.props.filterIcon} afterInput={iconsNode}/>
             {this._renderPopup()}
           </div>
           {this.props.error && (<div className={classNames(inputStyles.errorText, inputStyles[`size${this.props.size}`])}>

package/components/tab-trap/tab-trap.js

@@ -55,16 +55,11 @@ const TabTrap = forwardRef(function TabTrap({ children, trapDisabled = false, au
         if (previousFocusedNode instanceof HTMLElement &&
             previousFocusedNode.focus &&
             isNodeInVisiblePartOfPage(previousFocusedNode)) {
-            // If no delay is added, restoring focus caused by pressing Enter will trigger
-            // the onClick event on the previousFocusedNode, e.g.
-            // https://youtrack.jetbrains.com/issue/RG-2450/Anchor-should-be-focused-after-closing-datepicker#focus=Comments-27-10044234.0-0.
-            setTimeout(() => {
-                // This is to prevent the focus from being restored the first time
-                // componentWillUnmount is called in StrictMode.
-                if (!mountedRef.current) {
-                    previousFocusedNode.focus({ preventScroll: true });
-                }
-            });
+            // This is to prevent the focus from being restored the first time
+            // componentWillUnmount is called in StrictMode.
+            if (!mountedRef.current) {
+                previousFocusedNode.focus({ preventScroll: true });
+            }
         }
     }
     function focusElement(first = true) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jetbrains/ring-ui",
-  "version": "6.0.41",
+  "version": "6.0.43",
   "description": "JetBrains UI library",
   "author": "JetBrains",
   "license": "Apache-2.0",
@@ -88,16 +88,16 @@
     "@rollup/plugin-json": "^6.1.0",
     "@rollup/plugin-node-resolve": "^15.2.3",
     "@rollup/plugin-replace": "^5.0.7",
-    "@storybook/addon-a11y": "8.1.10",
-    "@storybook/addon-docs": "8.1.10",
-    "@storybook/addon-essentials": "8.1.10",
-    "@storybook/components": "8.1.10",
-    "@storybook/manager-api": "8.1.10",
-    "@storybook/preview-api": "8.1.10",
-    "@storybook/react": "8.1.10",
-    "@storybook/react-webpack5": "8.1.10",
-    "@storybook/test-runner": "^0.18.2",
-    "@storybook/theming": "8.1.10",
+    "@storybook/addon-a11y": "8.2.0",
+    "@storybook/addon-docs": "8.2.0",
+    "@storybook/addon-essentials": "8.2.0",
+    "@storybook/components": "8.2.0",
+    "@storybook/manager-api": "8.2.0",
+    "@storybook/preview-api": "8.2.0",
+    "@storybook/react": "8.2.0",
+    "@storybook/react-webpack5": "8.2.0",
+    "@storybook/test-runner": "^0.19.0",
+    "@storybook/theming": "8.2.0",
     "@testing-library/dom": "^10.3.1",
     "@testing-library/react": "^16.0.0",
     "@testing-library/user-event": "^14.5.2",
@@ -112,14 +112,14 @@
     "@types/sinon": "^17.0.3",
     "@types/sinon-chai": "^3.2.12",
     "@types/webpack-env": "^1.18.5",
-    "@typescript-eslint/eslint-plugin": "^7.15.0",
-    "@typescript-eslint/parser": "^7.15.0",
+    "@typescript-eslint/eslint-plugin": "^7.16.0",
+    "@typescript-eslint/parser": "^7.16.0",
     "@vitejs/plugin-react": "^4.3.1",
     "@wojtekmaj/enzyme-adapter-react-17": "^0.8.0",
     "acorn": "^8.12.1",
     "axe-playwright": "^2.0.1",
     "babel-plugin-require-context-hook": "^1.0.0",
-    "caniuse-lite": "^1.0.30001640",
+    "caniuse-lite": "^1.0.30001641",
     "chai": "^5.1.1",
     "chai-as-promised": "^8.0.0",
     "chai-dom": "^1.10.0",
@@ -137,7 +137,7 @@
     "eslint-plugin-react-hooks": "^4.6.2",
     "eslint-plugin-storybook": "^0.8.0",
     "events": "^3.3.0",
-    "glob": "^10.4.2",
+    "glob": "^11.0.0",
     "html-webpack-plugin": "^5.6.0",
     "http-server": "^14.1.1",
     "husky": "^9.0.11",
@@ -155,21 +155,21 @@
     "react-dom": "^18.3.1",
     "react-test-renderer": "^18.3.1",
     "regenerator-runtime": "^0.14.1",
-    "rimraf": "^5.0.7",
-    "rollup": "^4.18.0",
+    "rimraf": "^6.0.0",
+    "rollup": "^4.18.1",
     "rollup-plugin-clear": "^2.0.7",
     "rollup-plugin-styles": "^4.0.0",
     "sinon": "^18.0.0",
     "sinon-chai": "^3.7.0",
     "storage-mock": "^2.1.0",
-    "storybook": "8.1.10",
+    "storybook": "8.2.0",
     "storybook-addon-themes": "^6.1.0",
     "stylelint": "^16.6.1",
     "svg-inline-loader": "^0.8.2",
     "teamcity-service-messages": "^0.1.14",
     "terser-webpack-plugin": "^5.3.10",
     "typescript": "~5.5.3",
-    "vitest": "^1.6.0",
+    "vitest": "^2.0.1",
     "vitest-teamcity-reporter": "^0.3.0",
     "wallaby-webpack": "^3.9.16",
     "webpack": "^5.92.1",
@@ -210,7 +210,7 @@
     "@ungap/url-search-params": "^0.2.2",
     "babel-loader": "9.1.3",
     "babel-plugin-transform-define": "^2.1.4",
-    "browserslist": "^4.23.1",
+    "browserslist": "^4.23.2",
     "change-case": "^4.1.1",
     "classnames": "^2.5.1",
     "combokeys": "^3.0.1",
@@ -232,7 +232,7 @@
     "postcss-font-family-system-ui": "^5.0.0",
     "postcss-loader": "^8.1.1",
     "postcss-modules-values-replace": "^4.2.0",
-    "postcss-preset-env": "^9.5.15",
+    "postcss-preset-env": "^9.6.0",
     "prop-types": "^15.8.1",
     "react-movable": "^3.2.0",
     "react-virtualized": "^9.22.5",