@skyscanner/backpack-web 33.3.1 → 34.0.0-premajor

package/bpk-react-utils/src/withDefaultProps.js

@@ -35,7 +35,10 @@ const withDefaultProps = (WrappedComponent, defaultProps) => {
     if (innerClassName) {
       classNames.push(innerClassName);
     }
-    return /*#__PURE__*/_jsx(WrappedComponent, {
+    return /*#__PURE__*/_jsx(WrappedComponent
+    // General HOC difficult/impossible to remove
+    // eslint-disable-next-line @skyscanner/rules/forbid-component-props
+    , {
       className: classNames.join(' '),
       ...defaultRest,
       ...rest,

package/bpk-scrim-utils/src/withScrim.js

@@ -16,17 +16,17 @@
  * limitations under the License.
  */
 
+import { Component } from 'react';
+
 // @ts-expect-error Untyped import. See `decisions/imports-ts-suppressions.md`.
 import focusScope from 'a11y-focus-scope';
 // @ts-expect-error Untyped import. See `decisions/imports-ts-suppressions.md`.
 import focusStore from 'a11y-focus-store';
-import { Component } from 'react';
 import { cssModules, isDeviceIpad, isDeviceIphone, wrapDisplayName } from "../../bpk-react-utils";
 import BpkScrim from "./BpkScrim";
 import { fixBody, lockScroll, restoreScroll, storeScroll, unfixBody, unlockScroll } from "./scroll-utils";
 import STYLES from "./bpk-scrim-content.module.css";
-import { jsx as _jsx } from "react/jsx-runtime";
-import { jsxs as _jsxs } from "react/jsx-runtime";
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 const getClassName = cssModules(STYLES);
 const withScrim = WrappedComponent => {
   class WithScrim extends Component {

package/bpk-scrim-utils/src/withScrimmedPortal.js

@@ -18,8 +18,8 @@
 
 import { useState, useEffect } from 'react';
 import { createPortal } from 'react-dom';
-import withScrim from "./withScrim";
 import BpkScrim from "./BpkScrim";
+import withScrim from "./withScrim";
 import { jsx as _jsx } from "react/jsx-runtime";
 const getPortalElement = target => {
   const portalElement = target && typeof target === 'function' ? target() : null;

package/bpk-theming/src/BpkThemeProvider.js

@@ -16,8 +16,8 @@
  * limitations under the License.
  */
 
-import { Component } from 'react';
 import PropTypes from 'prop-types';
+import { Component } from 'react';
 import { jsx as _jsx } from "react/jsx-runtime";
 const uniq = (arr = []) => {
   const seen = {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyscanner/backpack-web",
-  "version": "33.3.1",
+  "version": "34.0.0-premajor",
   "description": "Backpack Design System web library",
   "repository": {
     "type": "git",
@@ -22,9 +22,11 @@
     "access": "public"
   },
   "dependencies": {
-    "@popperjs/core": "^2.11.5",
+    "@floating-ui/dom": "^1.6.3",
+    "@floating-ui/react": "^0.26.12",
+    "@popperjs/core": "^2.11.8",
     "@react-google-maps/api": "^2.12.0",
-    "@skyscanner/bpk-foundations-web": "^17.5.3",
+    "@skyscanner/bpk-foundations-web": "^17.9.0",
     "@skyscanner/bpk-svgs": "^19.1.0",
     "a11y-focus-scope": "^1.1.3",
     "a11y-focus-store": "^1.0.0",

package/unstable__bpk-mixins/_chips.scss

@@ -237,4 +237,13 @@
     color: tokens.$bpk-text-disabled-day;
     cursor: not-allowed;
   }
+
+  &--icon-only {
+    padding-inline-end: tokens.bpk-spacing-md();
+    padding-inline-start: tokens.bpk-spacing-md();
+
+    &__leading-accessory-view {
+      @include margins.bpk-margin-trailing(0);
+    }
+  }
 }

package/bpk-component-button/src/commonButtonTests.js

@@ -1,133 +0,0 @@
-/*
- * Backpack - Skyscanner's Design System
- *
- * Copyright 2016 Skyscanner Ltd
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *   http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-import { render } from '@testing-library/react';
-import { jsx as _jsx } from "react/jsx-runtime";
-const commonButtonTests = ButtonToTest => {
-  describe('ButtonToTest', () => {
-    it('should render correctly', () => {
-      const {
-        asFragment
-      } = render( /*#__PURE__*/_jsx(ButtonToTest, {
-        children: "My button"
-      }));
-      expect(asFragment()).toMatchSnapshot();
-    });
-    it('should render correctly with a "href" attribute', () => {
-      const {
-        asFragment
-      } = render( /*#__PURE__*/_jsx(ButtonToTest, {
-        href: "#",
-        children: "My button"
-      }));
-      expect(asFragment()).toMatchSnapshot();
-    });
-    it('should render correctly with a "disabled" attribute', () => {
-      const {
-        asFragment
-      } = render( /*#__PURE__*/_jsx(ButtonToTest, {
-        disabled: true,
-        children: "My button"
-      }));
-      expect(asFragment()).toMatchSnapshot();
-    });
-    it('should render correctly with a "large" attribute', () => {
-      const {
-        asFragment
-      } = render( /*#__PURE__*/_jsx(ButtonToTest, {
-        large: true,
-        children: "My button"
-      }));
-      expect(asFragment()).toMatchSnapshot();
-    });
-    it('should render correctly with an "iconOnly" attribute', () => {
-      const {
-        asFragment
-      } = render( /*#__PURE__*/_jsx(ButtonToTest, {
-        iconOnly: true,
-        children: "My button"
-      }));
-      expect(asFragment()).toMatchSnapshot();
-    });
-    it('should respect the class names entered as a string', () => {
-      const {
-        asFragment
-      } = render( /*#__PURE__*/_jsx(ButtonToTest, {
-        large: true,
-        className: "custom-class-1 custom-class-2",
-        children: "My button"
-      }));
-      expect(asFragment()).toMatchSnapshot();
-    });
-    it('should add only bpk specific classes if className prop is set to empty string', () => {
-      const {
-        asFragment
-      } = render( /*#__PURE__*/_jsx(ButtonToTest, {
-        large: true,
-        className: "",
-        children: "My button"
-      }));
-      expect(asFragment()).toMatchSnapshot();
-    });
-    it('should render correctly with "blank" attribute', () => {
-      const {
-        asFragment
-      } = render( /*#__PURE__*/_jsx(ButtonToTest, {
-        href: "#",
-        blank: true,
-        children: "My button"
-      }));
-      expect(asFragment()).toMatchSnapshot();
-    });
-    it('should render correctly with "rel" attribute', () => {
-      const {
-        asFragment
-      } = render( /*#__PURE__*/_jsx(ButtonToTest, {
-        href: "#",
-        rel: "rel-attr",
-        children: "My button"
-      }));
-      expect(asFragment()).toMatchSnapshot();
-    });
-    it('should render correctly with "blank" and "rel" attributes', () => {
-      const {
-        asFragment
-      } = render( /*#__PURE__*/_jsx(ButtonToTest, {
-        href: "#",
-        blank: true,
-        rel: "rel-overwrite",
-        children: "My button"
-      }));
-      expect(asFragment()).toMatchSnapshot();
-    });
-    it('should render correctly with "disabled" and "href" attributes', () => {
-      const {
-        asFragment
-      } = render( /*#__PURE__*/_jsx(ButtonToTest, {
-        href: "#",
-        disabled: true,
-        children: "My button"
-      }));
-      expect(asFragment()).toMatchSnapshot();
-    });
-  });
-};
-
-// We export these to reuse them in other tests for buttons.
-// eslint-disable-next-line jest/no-export
-export default commonButtonTests;