@gitlab/ui 64.6.1 → 64.6.2

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [64.6.2](https://gitlab.com/gitlab-org/gitlab-ui/compare/v64.6.1...v64.6.2) (2023-06-13)
+
+
+### Bug Fixes
+
+* **dropdowns:** fix ResizeObserver loop error ([a4797a5](https://gitlab.com/gitlab-org/gitlab-ui/commit/a4797a5624505d6d4aa2d539fd589bbd2b610aee))
+
 ## [64.6.1](https://gitlab.com/gitlab-org/gitlab-ui/compare/v64.6.0...v64.6.1) (2023-06-08)
 
 

package/dist/components/base/new_dropdowns/base_dropdown/base_dropdown.js

@@ -137,6 +137,7 @@ var script = {
   },
   data() {
     return {
+      openedYet: false,
       visible: false,
       baseDropdownId: uniqueId('base-dropdown-')
     };
@@ -207,8 +208,8 @@ var script = {
       return {
         'gl-display-block!': this.visible,
         [FIXED_WIDTH_CLASS]: !this.fluidWidth,
-        'gl-fixed': this.isFixed,
-        'gl-absolute': !this.isFixed
+        'gl-fixed': this.openedYet && this.isFixed,
+        'gl-absolute': this.openedYet && !this.isFixed
       };
     },
     isFixed() {
@@ -271,26 +272,22 @@ var script = {
         childList: true,
         subtree: true
       });
-      await new Promise(resolve => {
-        const stopAutoUpdate = autoUpdate(this.toggleElement, this.$refs.content, async () => {
-          const {
-            x,
-            y
-          } = await computePosition(this.toggleElement, this.$refs.content, this.floatingUIConfig);
+      this.stopAutoUpdate = autoUpdate(this.toggleElement, this.$refs.content, async () => {
+        const {
+          x,
+          y
+        } = await computePosition(this.toggleElement, this.$refs.content, this.floatingUIConfig);
 
-          /**
-           * Due to the asynchronous nature of computePosition, it's technically possible for the
-           * component to have been destroyed by the time the promise resolves. In such case, we exit
-           * early to prevent a TypeError.
-           */
-          if (!this.$refs.content) return;
-          Object.assign(this.$refs.content.style, {
-            left: `${x}px`,
-            top: `${y}px`
-          });
-          resolve(stopAutoUpdate);
+        /**
+         * Due to the asynchronous nature of computePosition, it's technically possible for the
+         * component to have been destroyed by the time the promise resolves. In such case, we exit
+         * early to prevent a TypeError.
+         */
+        if (!this.$refs.content) return;
+        Object.assign(this.$refs.content.style, {
+          left: `${x}px`,
+          top: `${y}px`
         });
-        this.stopAutoUpdate = stopAutoUpdate;
       });
     },
     stopFloating() {
@@ -303,13 +300,13 @@ var script = {
       if (this.visible) {
         // The dropdown needs to be actually visible before we compute its position with Floating UI.
         await this.$nextTick();
-
+        this.openedYet = true;
         /**
          * We wait until the dropdown's position has been computed before emitting the `shown` event.
          * This ensures that, if the parent component attempts to focus an inner element, the dropdown
          * is already properly placed in the page. Otherwise, the page would scroll back to the top.
          */
-        await this.startFloating();
+        this.startFloating();
         this.$emit(GL_DROPDOWN_SHOWN);
       } else {
         this.stopFloating();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gitlab/ui",
-  "version": "64.6.1",
+  "version": "64.6.2",
   "description": "GitLab UI Components",
   "license": "MIT",
   "main": "dist/index.js",
@@ -84,13 +84,13 @@
   },
   "devDependencies": {
     "@arkweid/lefthook": "0.7.7",
-    "@babel/core": "^7.22.1",
-    "@babel/preset-env": "^7.22.4",
-    "@babel/preset-react": "^7.22.3",
+    "@babel/core": "^7.22.5",
+    "@babel/preset-env": "^7.22.5",
+    "@babel/preset-react": "^7.22.5",
     "@gitlab/eslint-plugin": "19.0.0",
     "@gitlab/fonts": "^1.2.0",
     "@gitlab/stylelint-config": "4.1.0",
-    "@gitlab/svgs": "3.51.0",
+    "@gitlab/svgs": "3.53.0",
     "@rollup/plugin-commonjs": "^11.1.0",
     "@rollup/plugin-node-resolve": "^7.1.3",
     "@rollup/plugin-replace": "^2.3.2",
@@ -117,9 +117,9 @@
     "babel-loader": "^8.0.5",
     "babel-plugin-require-context-hook": "^1.0.0",
     "bootstrap": "4.6.2",
-    "cypress": "12.13.0",
+    "cypress": "12.14.0",
     "emoji-regex": "^10.0.0",
-    "eslint": "8.41.0",
+    "eslint": "8.42.0",
     "eslint-import-resolver-jest": "3.0.2",
     "eslint-plugin-cypress": "2.13.3",
     "eslint-plugin-storybook": "0.6.12",

package/src/components/base/new_dropdowns/base_dropdown/base_dropdown.spec.js

@@ -163,6 +163,7 @@ describe('base dropdown', () => {
           buildWrapper();
 
           await findDefaultDropdownToggle().trigger('click');
+          await nextTick();
 
           expect(computePosition).toHaveBeenCalledWith(
             findDefaultDropdownToggle().element,
@@ -180,6 +181,7 @@ describe('base dropdown', () => {
           });
 
           await findDefaultDropdownToggle().trigger('click');
+          await nextTick();
 
           expect(computePosition).toHaveBeenCalledWith(
             findDefaultDropdownToggle().element,

package/src/components/base/new_dropdowns/base_dropdown/base_dropdown.vue

@@ -149,6 +149,7 @@ export default {
   },
   data() {
     return {
+      openedYet: false,
       visible: false,
       baseDropdownId: uniqueId('base-dropdown-'),
     };
@@ -221,8 +222,8 @@ export default {
       return {
         'gl-display-block!': this.visible,
         [FIXED_WIDTH_CLASS]: !this.fluidWidth,
-        'gl-fixed': this.isFixed,
-        'gl-absolute': !this.isFixed,
+        'gl-fixed': this.openedYet && this.isFixed,
+        'gl-absolute': this.openedYet && !this.isFixed,
       };
     },
     isFixed() {
@@ -290,29 +291,24 @@ export default {
         subtree: true,
       });
 
-      await new Promise((resolve) => {
-        const stopAutoUpdate = autoUpdate(this.toggleElement, this.$refs.content, async () => {
-          const { x, y } = await computePosition(
-            this.toggleElement,
-            this.$refs.content,
-            this.floatingUIConfig
-          );
-
-          /**
-           * Due to the asynchronous nature of computePosition, it's technically possible for the
-           * component to have been destroyed by the time the promise resolves. In such case, we exit
-           * early to prevent a TypeError.
-           */
-          if (!this.$refs.content) return;
+      this.stopAutoUpdate = autoUpdate(this.toggleElement, this.$refs.content, async () => {
+        const { x, y } = await computePosition(
+          this.toggleElement,
+          this.$refs.content,
+          this.floatingUIConfig
+        );
 
-          Object.assign(this.$refs.content.style, {
-            left: `${x}px`,
-            top: `${y}px`,
-          });
+        /**
+         * Due to the asynchronous nature of computePosition, it's technically possible for the
+         * component to have been destroyed by the time the promise resolves. In such case, we exit
+         * early to prevent a TypeError.
+         */
+        if (!this.$refs.content) return;
 
-          resolve(stopAutoUpdate);
+        Object.assign(this.$refs.content.style, {
+          left: `${x}px`,
+          top: `${y}px`,
         });
-        this.stopAutoUpdate = stopAutoUpdate;
       });
     },
     stopFloating() {
@@ -325,13 +321,13 @@ export default {
       if (this.visible) {
         // The dropdown needs to be actually visible before we compute its position with Floating UI.
         await this.$nextTick();
-
+        this.openedYet = true;
         /**
          * We wait until the dropdown's position has been computed before emitting the `shown` event.
          * This ensures that, if the parent component attempts to focus an inner element, the dropdown
          * is already properly placed in the page. Otherwise, the page would scroll back to the top.
          */
-        await this.startFloating();
+        this.startFloating();
 
         this.$emit(GL_DROPDOWN_SHOWN);
       } else {