From 9c26e834b703b44a0e300dbbee4e706080a0289c Mon Sep 17 00:00:00 2001
From: Adrien Maston <adrien@37signals.com>
Date: Wed, 4 Mar 2026 11:30:42 +0100
Subject: [PATCH 1/2] Don't try to orient column menus

---
 app/javascript/controllers/dialog_controller.js | 7 +++++--
 app/views/boards/show/menu/_column.html.erb     | 2 +-
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/app/javascript/controllers/dialog_controller.js b/app/javascript/controllers/dialog_controller.js
index b52eb2ea6..1790ac471 100644
--- a/app/javascript/controllers/dialog_controller.js
+++ b/app/javascript/controllers/dialog_controller.js
@@ -7,7 +7,8 @@ export default class extends Controller {
   static values = {
     modal: { type: Boolean, default: false },
     sizing: { type: Boolean, default: true },
-    autoOpen: { type: Boolean, default: false }
+    autoOpen: { type: Boolean, default: false },
+    orient: { type: Boolean, default: true }
   }
 
   connect() {
@@ -26,7 +27,9 @@ export default class extends Controller {
       this.dialogTarget.showModal()
     } else {
       this.dialogTarget.show()
-      orient({ target: this.dialogTarget, anchor: this.element })
+      if (this.orientValue) {
+        orient({ target: this.dialogTarget, anchor: this.element })
+      }
     }
 
     this.loadLazyFrames()
diff --git a/app/views/boards/show/menu/_column.html.erb b/app/views/boards/show/menu/_column.html.erb
index ebfdd3855..10c751bca 100644
--- a/app/views/boards/show/menu/_column.html.erb
+++ b/app/views/boards/show/menu/_column.html.erb
@@ -1,4 +1,4 @@
-<div id="<%= dom_id(column, :menu) %>" class="cards__menu" data-controller="dialog" data-action="keydown.esc->dialog#close click@document->dialog#closeOnClickOutside">
+<div id="<%= dom_id(column, :menu) %>" class="cards__menu" data-controller="dialog" data-dialog-orient-value="false" data-action="keydown.esc->dialog#close click@document->dialog#closeOnClickOutside">
   <button class="btn btn--circle txt-x-small borderless" data-action="click->dialog#open">
     <%= icon_tag "menu-dots-horizontal", class: "translucent" %>
     <span class="for-screen-reader">Column options</span>

From 920502b5907a8341919e8b1f510c114818312c0b Mon Sep 17 00:00:00 2001
From: Adrien Maston <adrien@37signals.com>
Date: Wed, 4 Mar 2026 11:31:45 +0100
Subject: [PATCH 2/2] Scroll column into view when opening its menu

---
 .../controllers/collapsible_columns_controller.js           | 6 ++++++
 app/views/boards/show/_column.html.erb                      | 2 +-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/app/javascript/controllers/collapsible_columns_controller.js b/app/javascript/controllers/collapsible_columns_controller.js
index 35c5efedf..b3c429140 100644
--- a/app/javascript/controllers/collapsible_columns_controller.js
+++ b/app/javascript/controllers/collapsible_columns_controller.js
@@ -54,6 +54,12 @@ export default class extends Controller {
     }
   }
 
+  frameColumnOnMobile(event) {
+    if (!this.#isDesktop) {
+      event.currentTarget.scrollIntoView({ behavior: "smooth", inline: "center" })
+    }
+  }
+
   async #restoreColumnsDisablingTransitions() {
     this.#disableTransitions()
     this.#restoreColumns()
diff --git a/app/views/boards/show/_column.html.erb b/app/views/boards/show/_column.html.erb
index b5bba3339..e99143c83 100644
--- a/app/views/boards/show/_column.html.erb
+++ b/app/views/boards/show/_column.html.erb
@@ -4,7 +4,7 @@
     drag_and_strum_target: "container",
     collapsible_columns_target: "column",
     controller: "clicker",
-    action: "turbo:before-stream-render@document->collapsible-columns#restoreState focus->collapsible-columns#focusOnColumn"
+    action: "turbo:before-stream-render@document->collapsible-columns#restoreState focus->collapsible-columns#focusOnColumn dialog:show->collapsible-columns#frameColumnOnMobile"
   } do %>
   <header class="cards__header">
     <%= render "boards/show/menu/column", column: column %>