Rough swap of popperjs with floating-ui

#872

Author: dancormier

Gruntfile.js

@@ -88,7 +88,7 @@ module.exports = function(grunt) {
                 files: {
                     'dist/js/stacks.js': [
                         'node_modules/stimulus/dist/stimulus.umd.js',
-                        'node_modules/@popperjs/core/dist/umd/popper.js',
+                        'node_modules/@floating-ui/core/dist/floating-ui.core.js',
                         'build/lib/ts/stacks.js',
                         'build/lib/ts/controllers/**/*.js',
                         'build/lib/ts/finalize.js'

lib/ts/controllers/s-popover.ts

@@ -4,7 +4,7 @@ namespace Stacks {
 
     export abstract class BasePopoverController extends StacksController {
         // @ts-ignore
-        private popper!: Popper;
+        private floatingUI!: FloatingUICore;
 
         protected popoverElement!: HTMLElement;
 
@@ -71,7 +71,7 @@ namespace Stacks {
             this.validate();
             if (this.isVisible) {
                 // just call initialize here, not show. This keeps already visible popovers from adding/firing document events
-                this.initializePopper();
+                this.initializeFloatingUI();
             } else if (this.data.get("auto-show") === "true") {
                 this.show(null);
             }
@@ -84,9 +84,9 @@ namespace Stacks {
          */
         disconnect() {
             this.hide();
-            if (this.popper) {
-                this.popper.destroy();
-                delete this.popper;
+            if (this.floatingUI) {
+                this.floatingUI.destroy();
+                delete this.floatingUI;
             }
             super.disconnect();
         }
@@ -110,8 +110,8 @@ namespace Stacks {
                 dispatcher: dispatcherElement
             }).defaultPrevented) { return; }
 
-            if (!this.popper) {
-                this.initializePopper();
+            if (!this.floatingUI) {
+                this.initializeFloatingUI();
             }
 
             this.popoverElement!.classList.add("is-visible");
@@ -136,10 +136,10 @@ namespace Stacks {
 
             this.popoverElement.classList.remove("is-visible");
 
-            if (this.popper) {
+            if (this.floatingUI) {
                 // completely destroy the popper on hide; this is in line with Popper.js's performance recommendations
-                this.popper.destroy();
-                delete this.popper;
+                this.floatingUI.destroy();
+                delete this.floatingUI;
             }
 
             // on first interaction, hide-on-outside-click with value "after-dismissal" reverts to the default behavior
@@ -180,25 +180,29 @@ namespace Stacks {
         /**
          * Initializes the Popper for this instance
          */
-        private initializePopper() {
+        private initializeFloatingUI() {
             // @ts-ignore
-            this.popper = Popper.createPopper(this.referenceElement, this.popoverElement, {
-                placement: this.data.get("placement") || "bottom",
-                modifiers: [
-                    {
-                        name: "offset",
-                        options: {
-                            offset: [0, 10], // The entire popover should be 10px away from the element
-                        }
-                    },
-                    {
-                        name: "arrow",
-                        options: {
-                            element: ".s-popover--arrow"
-                        },
-                    },
-                ]
-            });
+            // this.floatingUI = FloatingUICore.computePosition(this.refrenceElement, this.popoverElement, {
+            // // @ts-ignore
+            // middleware: [ FloatingUICore.offset(), FloatingUICore.flip(), FloatingUICore.shift() ]
+            // });
+            // this.floatingUI = FloatingUICore.createPopper(this.referenceElement, this.popoverElement, {
+            //     placement: this.data.get("placement") || "bottom",
+            //     modifiers: [
+            //         {
+            //             name: "offset",
+            //             options: {
+            //                 offset: [0, 10], // The entire popover should be 10px away from the element
+            //             }
+            //         },
+            //         {
+            //             name: "arrow",
+            //             options: {
+            //                 element: ".s-popover--arrow"
+            //             },
+            //         },
+            //     ]
+            // });
         }
 
         /**
@@ -262,8 +266,8 @@ namespace Stacks {
          * Schedules the popover to update on the next animation frame if visible
          */
         protected scheduleUpdate() {
-            if (this.popper && this.isVisible) {
-                this.popper.update();
+            if (this.floatingUI && this.isVisible) {
+                this.floatingUI.update();
             }
         }
     }

package-lock.json

@@ -19,7 +19,7 @@
   },
   "license": "MIT",
   "dependencies": {
-    "@popperjs/core": "^2.11.3",
+    "@floating-ui/dom": "^0.4.1",
     "stimulus": "^2.0.0"
   },
   "devDependencies": {