Merge branch 'allow-to-pass-strategy-option-to-dropdown-menu' of github.com:lsouoliveira/ruby_ui into allow-to-pass-strategy-option-to-dropdown-menu

Author: lsouoliveira

lib/ruby_ui/dropdown_menu/dropdown_menu.rb

@@ -17,7 +17,7 @@ def default_attrs
       {
         class: [
           "group/dropdown-menu",
-          (@strategy == "absolute") ? "is-absolute" : "is-fixed"
+          (strategy == "absolute") ? "is-absolute" : "is-fixed"
         ],
         data: {
           controller: "ruby-ui--dropdown-menu",
@@ -26,5 +26,9 @@ def default_attrs
         }
       }
     end
+
+    def strategy
+      @_strategy ||= @options[:strategy] || "absolute"
+    end
   end
 end

lib/ruby_ui/dropdown_menu/dropdown_menu_controller.js

@@ -1,5 +1,11 @@
 import { Controller } from "@hotwired/stimulus";
-import { computePosition, flip, shift, offset, autoUpdate } from "@floating-ui/dom";
+import {
+  computePosition,
+  flip,
+  shift,
+  offset,
+  autoUpdate,
+} from "@floating-ui/dom";
 
 export default class extends Controller {
   static targets = ["trigger", "content", "menuItem"];
@@ -17,15 +23,17 @@ export default class extends Controller {
   connect() {
     this.boundHandleKeydown = this.#handleKeydown.bind(this); // Bind the function so we can remove it later
     this.selectedIndex = -1;
-    this.cleanup = autoUpdate(
+    this.autoUpdateCleanup = autoUpdate(
       this.triggerTarget,
       this.contentTarget,
       this.#computeTooltip.bind(this),
     );
   }
 
   disconnect() {
-    this.cleanup();
+    if (this.autoUpdateCleanup) {
+      this.autoUpdateCleanup();
+    }
   }
 
   #computeTooltip() {