@atlaskit/react-select 2.4.3 → 2.4.4

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @atlaskit/react-select
 
+## 2.4.4
+
+### Patch Changes
+
+- [#142181](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/pull-requests/142181)
+  [`d6934885b334e`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/d6934885b334e) -
+  Fixed some new icons that were enabled with the incorrect feature flag.
+
 ## 2.4.3
 
 ### Patch Changes

package/dist/cjs/compiled/components/multi-value.js

@@ -18,7 +18,6 @@ var _react2 = require("@compiled/react");
 var _selectClear = _interopRequireDefault(require("@atlaskit/icon/glyph/select-clear"));
 var _cross = _interopRequireDefault(require("@atlaskit/icon/utility/cross"));
 var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
-var _compiled = require("@atlaskit/primitives/compiled");
 var _utils = require("../../utils");
 function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(e) { return e ? t : r; })(e); }
 function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != _typeof(e) && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && {}.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
@@ -79,34 +78,6 @@ var MultiValueLabel = exports.MultiValueLabel = function MultiValueLabel(_ref2)
 };
 var disabledStyles = null;
 var enabledStyles = null;
-var iconWrapperStyles = {
-  root: "_ca0qv77o _u5f3v77o _n3tdv77o _19bvv77o"
-};
-var renderIcon = function renderIcon() {
-  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
-  if ((0, _platformFeatureFlags.fg)('platform-component-visual-refresh')) {
-    return /*#__PURE__*/React.createElement(_cross.default, {
-      label: "",
-      color: "currentColor"
-    });
-  }
-
-  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
-  if ((0, _platformFeatureFlags.fg)('platform-visual-refresh-icons-legacy-facade')) {
-    return /*#__PURE__*/React.createElement(_compiled.Inline, {
-      xcss: iconWrapperStyles.root
-    }, /*#__PURE__*/React.createElement(_cross.default, {
-      label: "",
-      color: "currentColor"
-    }));
-  }
-  return /*#__PURE__*/React.createElement(_selectClear.default, {
-    label: "",
-    primaryColor: "transparent",
-    size: "small",
-    secondaryColor: "inherit"
-  });
-};
 function MultiValueRemove(_ref3) {
   var isDisabled = _ref3.isDisabled,
     isFocused = _ref3.isFocused,
@@ -119,7 +90,14 @@ function MultiValueRemove(_ref3) {
   }), /*#__PURE__*/React.createElement("div", {
     "data-testid": isDisabled ? 'hide-clear-icon' : 'show-clear-icon',
     className: (0, _runtime.ax)([isDisabled && "_1e0cglyw", !isDisabled && "_1e0c1kw7"])
-  }, renderIcon()));
+  }, /*#__PURE__*/React.createElement(_cross.default, {
+    label: "",
+    color: "currentColor",
+    LEGACY_fallbackIcon: _selectClear.default,
+    LEGACY_primaryColor: "transparent",
+    LEGACY_secondaryColor: "inherit",
+    LEGACY_size: "small"
+  })));
 }
 var MultiValue = function MultiValue(props) {
   var children = props.children,

package/dist/cjs/components/internal/scroll-manager.js

@@ -40,7 +40,7 @@ function ScrollManager(_ref) {
   };
   return /*#__PURE__*/_react.default.createElement(_react.Fragment, null, lockEnabled &&
   /*#__PURE__*/
-  // eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions
+  // eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions, @atlassian/a11y/interactive-element-not-keyboard-focusable
   _react.default.createElement("div", {
     onClick: blurSelectInput,
     style: {

package/dist/cjs/emotion/components/internal/scroll-manager.js

@@ -51,7 +51,7 @@ function ScrollManager(_ref) {
     setScrollLockTarget(element);
   };
   return (0, _react2.jsx)(_react.Fragment, null, lockEnabled &&
-  // eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions
+  // eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions, @atlassian/a11y/interactive-element-not-keyboard-focusable
   (0, _react2.jsx)("div", {
     onClick: blurSelectInput,
     css: styles

package/dist/cjs/emotion/components/multi-value.js

@@ -12,7 +12,6 @@ var _react = require("@emotion/react");
 var _selectClear = _interopRequireDefault(require("@atlaskit/icon/glyph/select-clear"));
 var _cross = _interopRequireDefault(require("@atlaskit/icon/utility/cross"));
 var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
-var _primitives = require("@atlaskit/primitives");
 var _utils = require("../../utils");
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; } /**
@@ -138,34 +137,6 @@ var disabledStyles = (0, _react.css)({
 var enabledStyles = (0, _react.css)({
   display: 'inherit'
 });
-var iconWrapperStyles = (0, _primitives.xcss)({
-  padding: 'space.025'
-});
-var renderIcon = function renderIcon() {
-  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
-  if ((0, _platformFeatureFlags.fg)('platform-component-visual-refresh')) {
-    return (0, _react.jsx)(_cross.default, {
-      label: "",
-      color: "currentColor"
-    });
-  }
-
-  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
-  if ((0, _platformFeatureFlags.fg)('platform-visual-refresh-icons-legacy-facade')) {
-    return (0, _react.jsx)(_primitives.Inline, {
-      xcss: iconWrapperStyles
-    }, (0, _react.jsx)(_cross.default, {
-      label: "",
-      color: "currentColor"
-    }));
-  }
-  return (0, _react.jsx)(_selectClear.default, {
-    label: "",
-    primaryColor: "transparent",
-    size: "small",
-    secondaryColor: "inherit"
-  });
-};
 function MultiValueRemove(_ref5) {
   var isDisabled = _ref5.isDisabled,
     innerProps = _ref5.innerProps;
@@ -174,7 +145,14 @@ function MultiValueRemove(_ref5) {
     (0, _react.jsx)("div", innerProps, (0, _react.jsx)("div", {
       css: isDisabled ? disabledStyles : enabledStyles,
       "data-testid": isDisabled ? 'hide-clear-icon' : 'show-clear-icon'
-    }, renderIcon()))
+    }, (0, _react.jsx)(_cross.default, {
+      label: "",
+      color: "currentColor",
+      LEGACY_fallbackIcon: _selectClear.default,
+      LEGACY_primaryColor: "transparent",
+      LEGACY_secondaryColor: "inherit",
+      LEGACY_size: "small"
+    })))
   );
 }
 var MultiValue = function MultiValue(props) {

package/dist/es2019/compiled/components/multi-value.js

@@ -7,7 +7,6 @@ import { cx } from '@compiled/react';
 import LegacySelectClearIcon from '@atlaskit/icon/glyph/select-clear';
 import CrossIcon from '@atlaskit/icon/utility/cross';
 import { fg } from '@atlaskit/platform-feature-flags';
-import { Inline } from '@atlaskit/primitives/compiled';
 import { getStyleProps } from '../../utils';
 const multiValueStyles = {
   root: "_2rkogqwt _1e0c1txw _1ul9ze3t _19pkv77o _2hwxv77o _otyrv77o _18u0v77o _bfhkm7j4 _p12f1osq _syazjpor _3yq3glyw",
@@ -60,34 +59,6 @@ export const MultiValueLabel = ({
 };
 const disabledStyles = null;
 const enabledStyles = null;
-const iconWrapperStyles = {
-  root: "_ca0qv77o _u5f3v77o _n3tdv77o _19bvv77o"
-};
-const renderIcon = () => {
-  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
-  if (fg('platform-component-visual-refresh')) {
-    return /*#__PURE__*/React.createElement(CrossIcon, {
-      label: "",
-      color: "currentColor"
-    });
-  }
-
-  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
-  if (fg('platform-visual-refresh-icons-legacy-facade')) {
-    return /*#__PURE__*/React.createElement(Inline, {
-      xcss: iconWrapperStyles.root
-    }, /*#__PURE__*/React.createElement(CrossIcon, {
-      label: "",
-      color: "currentColor"
-    }));
-  }
-  return /*#__PURE__*/React.createElement(LegacySelectClearIcon, {
-    label: "",
-    primaryColor: "transparent",
-    size: "small",
-    secondaryColor: "inherit"
-  });
-};
 export function MultiValueRemove({
   isDisabled,
   isFocused,
@@ -101,7 +72,14 @@ export function MultiValueRemove({
   }), /*#__PURE__*/React.createElement("div", {
     "data-testid": isDisabled ? 'hide-clear-icon' : 'show-clear-icon',
     className: ax([isDisabled && "_1e0cglyw", !isDisabled && "_1e0c1kw7"])
-  }, renderIcon()));
+  }, /*#__PURE__*/React.createElement(CrossIcon, {
+    label: "",
+    color: "currentColor",
+    LEGACY_fallbackIcon: LegacySelectClearIcon,
+    LEGACY_primaryColor: "transparent",
+    LEGACY_secondaryColor: "inherit",
+    LEGACY_size: "small"
+  })));
 }
 const MultiValue = props => {
   const {

package/dist/es2019/components/internal/scroll-manager.js

@@ -30,7 +30,7 @@ export default function ScrollManager({
   };
   return /*#__PURE__*/React.createElement(Fragment, null, lockEnabled &&
   /*#__PURE__*/
-  // eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions
+  // eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions, @atlassian/a11y/interactive-element-not-keyboard-focusable
   React.createElement("div", {
     onClick: blurSelectInput,
     style: {

package/dist/es2019/emotion/components/internal/scroll-manager.js

@@ -43,7 +43,7 @@ export default function ScrollManager({
     setScrollLockTarget(element);
   };
   return jsx(Fragment, null, lockEnabled &&
-  // eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions
+  // eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions, @atlassian/a11y/interactive-element-not-keyboard-focusable
   jsx("div", {
     onClick: blurSelectInput,
     css: styles

package/dist/es2019/emotion/components/multi-value.js

@@ -8,7 +8,6 @@ import { css, jsx } from '@emotion/react';
 import LegacySelectClearIcon from '@atlaskit/icon/glyph/select-clear';
 import CrossIcon from '@atlaskit/icon/utility/cross';
 import { fg } from '@atlaskit/platform-feature-flags';
-import { Inline, xcss } from '@atlaskit/primitives';
 import { getStyleProps } from '../../utils';
 export const multiValueCSS = ({
   isDisabled,
@@ -130,34 +129,6 @@ const disabledStyles = css({
 const enabledStyles = css({
   display: 'inherit'
 });
-const iconWrapperStyles = xcss({
-  padding: 'space.025'
-});
-const renderIcon = () => {
-  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
-  if (fg('platform-component-visual-refresh')) {
-    return jsx(CrossIcon, {
-      label: "",
-      color: "currentColor"
-    });
-  }
-
-  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
-  if (fg('platform-visual-refresh-icons-legacy-facade')) {
-    return jsx(Inline, {
-      xcss: iconWrapperStyles
-    }, jsx(CrossIcon, {
-      label: "",
-      color: "currentColor"
-    }));
-  }
-  return jsx(LegacySelectClearIcon, {
-    label: "",
-    primaryColor: "transparent",
-    size: "small",
-    secondaryColor: "inherit"
-  });
-};
 export function MultiValueRemove({
   isDisabled,
   innerProps
@@ -167,7 +138,14 @@ export function MultiValueRemove({
     jsx("div", innerProps, jsx("div", {
       css: isDisabled ? disabledStyles : enabledStyles,
       "data-testid": isDisabled ? 'hide-clear-icon' : 'show-clear-icon'
-    }, renderIcon()))
+    }, jsx(CrossIcon, {
+      label: "",
+      color: "currentColor",
+      LEGACY_fallbackIcon: LegacySelectClearIcon,
+      LEGACY_primaryColor: "transparent",
+      LEGACY_secondaryColor: "inherit",
+      LEGACY_size: "small"
+    })))
   );
 }
 const MultiValue = props => {

package/dist/esm/compiled/components/multi-value.js

@@ -10,7 +10,6 @@ import { cx } from '@compiled/react';
 import LegacySelectClearIcon from '@atlaskit/icon/glyph/select-clear';
 import CrossIcon from '@atlaskit/icon/utility/cross';
 import { fg } from '@atlaskit/platform-feature-flags';
-import { Inline } from '@atlaskit/primitives/compiled';
 import { getStyleProps } from '../../utils';
 var multiValueStyles = {
   root: "_2rkogqwt _1e0c1txw _1ul9ze3t _19pkv77o _2hwxv77o _otyrv77o _18u0v77o _bfhkm7j4 _p12f1osq _syazjpor _3yq3glyw",
@@ -67,34 +66,6 @@ export var MultiValueLabel = function MultiValueLabel(_ref2) {
 };
 var disabledStyles = null;
 var enabledStyles = null;
-var iconWrapperStyles = {
-  root: "_ca0qv77o _u5f3v77o _n3tdv77o _19bvv77o"
-};
-var renderIcon = function renderIcon() {
-  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
-  if (fg('platform-component-visual-refresh')) {
-    return /*#__PURE__*/React.createElement(CrossIcon, {
-      label: "",
-      color: "currentColor"
-    });
-  }
-
-  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
-  if (fg('platform-visual-refresh-icons-legacy-facade')) {
-    return /*#__PURE__*/React.createElement(Inline, {
-      xcss: iconWrapperStyles.root
-    }, /*#__PURE__*/React.createElement(CrossIcon, {
-      label: "",
-      color: "currentColor"
-    }));
-  }
-  return /*#__PURE__*/React.createElement(LegacySelectClearIcon, {
-    label: "",
-    primaryColor: "transparent",
-    size: "small",
-    secondaryColor: "inherit"
-  });
-};
 export function MultiValueRemove(_ref3) {
   var isDisabled = _ref3.isDisabled,
     isFocused = _ref3.isFocused,
@@ -107,7 +78,14 @@ export function MultiValueRemove(_ref3) {
   }), /*#__PURE__*/React.createElement("div", {
     "data-testid": isDisabled ? 'hide-clear-icon' : 'show-clear-icon',
     className: ax([isDisabled && "_1e0cglyw", !isDisabled && "_1e0c1kw7"])
-  }, renderIcon()));
+  }, /*#__PURE__*/React.createElement(CrossIcon, {
+    label: "",
+    color: "currentColor",
+    LEGACY_fallbackIcon: LegacySelectClearIcon,
+    LEGACY_primaryColor: "transparent",
+    LEGACY_secondaryColor: "inherit",
+    LEGACY_size: "small"
+  })));
 }
 var MultiValue = function MultiValue(props) {
   var children = props.children,

package/dist/esm/components/internal/scroll-manager.js

@@ -30,7 +30,7 @@ export default function ScrollManager(_ref) {
   };
   return /*#__PURE__*/React.createElement(Fragment, null, lockEnabled &&
   /*#__PURE__*/
-  // eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions
+  // eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions, @atlassian/a11y/interactive-element-not-keyboard-focusable
   React.createElement("div", {
     onClick: blurSelectInput,
     style: {

package/dist/esm/emotion/components/internal/scroll-manager.js

@@ -43,7 +43,7 @@ export default function ScrollManager(_ref) {
     setScrollLockTarget(element);
   };
   return jsx(Fragment, null, lockEnabled &&
-  // eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions
+  // eslint-disable-next-line jsx-a11y/click-events-have-key-events, jsx-a11y/no-static-element-interactions, @atlassian/a11y/interactive-element-not-keyboard-focusable
   jsx("div", {
     onClick: blurSelectInput,
     css: styles

package/dist/esm/emotion/components/multi-value.js

@@ -11,7 +11,6 @@ import { css, jsx } from '@emotion/react';
 import LegacySelectClearIcon from '@atlaskit/icon/glyph/select-clear';
 import CrossIcon from '@atlaskit/icon/utility/cross';
 import { fg } from '@atlaskit/platform-feature-flags';
-import { Inline, xcss } from '@atlaskit/primitives';
 import { getStyleProps } from '../../utils';
 export var multiValueCSS = function multiValueCSS(_ref) {
   var isDisabled = _ref.isDisabled,
@@ -132,34 +131,6 @@ var disabledStyles = css({
 var enabledStyles = css({
   display: 'inherit'
 });
-var iconWrapperStyles = xcss({
-  padding: 'space.025'
-});
-var renderIcon = function renderIcon() {
-  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
-  if (fg('platform-component-visual-refresh')) {
-    return jsx(CrossIcon, {
-      label: "",
-      color: "currentColor"
-    });
-  }
-
-  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
-  if (fg('platform-visual-refresh-icons-legacy-facade')) {
-    return jsx(Inline, {
-      xcss: iconWrapperStyles
-    }, jsx(CrossIcon, {
-      label: "",
-      color: "currentColor"
-    }));
-  }
-  return jsx(LegacySelectClearIcon, {
-    label: "",
-    primaryColor: "transparent",
-    size: "small",
-    secondaryColor: "inherit"
-  });
-};
 export function MultiValueRemove(_ref5) {
   var isDisabled = _ref5.isDisabled,
     innerProps = _ref5.innerProps;
@@ -168,7 +139,14 @@ export function MultiValueRemove(_ref5) {
     jsx("div", innerProps, jsx("div", {
       css: isDisabled ? disabledStyles : enabledStyles,
       "data-testid": isDisabled ? 'hide-clear-icon' : 'show-clear-icon'
-    }, renderIcon()))
+    }, jsx(CrossIcon, {
+      label: "",
+      color: "currentColor",
+      LEGACY_fallbackIcon: LegacySelectClearIcon,
+      LEGACY_primaryColor: "transparent",
+      LEGACY_secondaryColor: "inherit",
+      LEGACY_size: "small"
+    })))
   );
 }
 var MultiValue = function MultiValue(props) {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/react-select",
-	"version": "2.4.3",
+	"version": "2.4.4",
 	"description": "A forked version of react-select to only be used in atlaskit/select",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -48,7 +48,7 @@
 		"react-dom": "^18.2.0"
 	},
 	"devDependencies": {
-		"@atlaskit/checkbox": "^17.0.0",
+		"@atlaskit/checkbox": "^17.1.0",
 		"@atlaskit/form": "^12.0.0",
 		"@atlassian/feature-flags-test-utils": "^0.3.0",
 		"@testing-library/react": "^13.4.0",
@@ -72,9 +72,6 @@
 		"design-system-select-fix-placement": {
 			"type": "boolean"
 		},
-		"platform-visual-refresh-icons-legacy-facade": {
-			"type": "boolean"
-		},
 		"platform_dst_layer_observer_select": {
 			"type": "boolean"
 		},