From 16c914c7c52fee26e78d2ab9a452bc73224123b3 Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Thu, 30 Oct 2025 09:32:19 -0500
Subject: [PATCH 01/12] Center system comments

---
 app/assets/stylesheets/comments.css | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/app/assets/stylesheets/comments.css b/app/assets/stylesheets/comments.css
index 33aacc74a..6720fa258 100644
--- a/app/assets/stylesheets/comments.css
+++ b/app/assets/stylesheets/comments.css
@@ -126,6 +126,7 @@
     }
 
     .comment__body {
+      padding: 0;
       text-align: center;
     }
 
@@ -135,6 +136,7 @@
         var(--color-canvas) 0 1px,
         var(--color-ink-lightest) 1px 10px
       );
+      padding-inline: var(--comment-padding-inline);
     }
 
     .reactions {

From f82f44e300c591b16123b176fd34fbe1adb29a3e Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Thu, 30 Oct 2025 10:05:02 -0500
Subject: [PATCH 02/12] We don't want this on the new card form

---
 app/views/cards/_container.html.erb | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/app/views/cards/_container.html.erb b/app/views/cards/_container.html.erb
index add4b0049..a8dd4dd76 100644
--- a/app/views/cards/_container.html.erb
+++ b/app/views/cards/_container.html.erb
@@ -20,7 +20,10 @@
             <%= render "cards/container/title", card: card %>
             <%= render "cards/display/perma/steps", card: card %>
           </div>
-          <%= render "cards/triage/columns", card: card %>
+
+          <% if card.published? %>
+            <%= render "cards/triage/columns", card: card %>
+          <% end %>
         </div>
 
         <footer class="card__footer full-width flex align-start gap">

From 17a33a9c81a92559d70e3d3f0c78ffef91657786 Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Thu, 30 Oct 2025 10:10:33 -0500
Subject: [PATCH 03/12] Avoid setting column on closed cards

---
 app/helpers/columns_helper.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/helpers/columns_helper.rb b/app/helpers/columns_helper.rb
index 793103aa7..56699d584 100644
--- a/app/helpers/columns_helper.rb
+++ b/app/helpers/columns_helper.rb
@@ -4,7 +4,7 @@ module ColumnsHelper
       tag.span(column.name, class: "overflow-ellipsis"),
       card_triage_path(card, column_id: column),
       method: :post,
-      class: [ "card__column-name btn", { "card__column-name--current": column == card.column } ],
+      class: [ "card__column-name btn", { "card__column-name--current": column == card.column && card.open? } ],
       form_class: "flex align-center gap-half",
       data: { turbo_frame: "_top" }
   end

From 5084952e3b5e8738c40b4c7ad89e9e1518df175c Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Thu, 30 Oct 2025 10:29:51 -0500
Subject: [PATCH 04/12] Don't animate restored column

---
 app/assets/stylesheets/card-columns.css | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/app/assets/stylesheets/card-columns.css b/app/assets/stylesheets/card-columns.css
index 7f64f7814..2e38b9447 100644
--- a/app/assets/stylesheets/card-columns.css
+++ b/app/assets/stylesheets/card-columns.css
@@ -208,6 +208,10 @@
       z-index: -1;
     }
 
+    .no-transitions &:after {
+      transition: none;
+    }
+
     .cards.is-collapsed & {
       block-size: 100%;
       flex-direction: column;

From 30d70429e9e0540976fd98c7d92c48218ed150f7 Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Thu, 30 Oct 2025 11:11:27 -0500
Subject: [PATCH 05/12] Fix wrapping

---
 app/views/my/menus/show.html.erb | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/app/views/my/menus/show.html.erb b/app/views/my/menus/show.html.erb
index ce16d93ee..3211d6002 100644
--- a/app/views/my/menus/show.html.erb
+++ b/app/views/my/menus/show.html.erb
@@ -23,9 +23,8 @@
 
   <%= render "filters/menu/hotkeys" %>
 
-  <div class="margin-block-start-half txt-x-small flex-inline align-center gap-half justify-center txt-subtle">
-    <span>BOXCAR is designed, built, and backed by</span>
-    <strong><%= icon_tag "37signals" %> <a href="https://37signals.com" target="_blank" class="txt-subtle">37signals</a>.</strong>
+  <div class="margin-block-start-half txt-x-small txt-align-center txt-subtle">
+    BOXCAR is designed, built, and backed by <strong class="txt-nowrap"><%= icon_tag "37signals" %> <a href="https://37signals.com" target="_blank" class="txt-subtle">37signals</a>.</strong>
   </div>
 
   <%= render "filters/menu/custom", filters: @user_filtering.filters %>

From cba945104a1728ffd5428abf8c61a4f90ef043c2 Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Thu, 30 Oct 2025 11:49:27 -0500
Subject: [PATCH 06/12] Make it harder to deselect yourself

---
 app/javascript/controllers/toggle_class_controller.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/app/javascript/controllers/toggle_class_controller.js b/app/javascript/controllers/toggle_class_controller.js
index 039ad2e65..357525f67 100644
--- a/app/javascript/controllers/toggle_class_controller.js
+++ b/app/javascript/controllers/toggle_class_controller.js
@@ -24,6 +24,7 @@ export default class extends Controller {
 
   checkNone() {
     this.checkboxTargets.forEach(checkbox => {
+      if (checkbox.dataset.collectionsFormTarget === "meCheckbox") return
       checkbox.checked = false
     })
   }

From 8d0041f6b9ab2955cc252da7ee35ca9e7bcc4026 Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Wed, 29 Oct 2025 19:34:33 -0500
Subject: [PATCH 07/12] Add a busy state for form buttons

---
 app/assets/stylesheets/buttons.css | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/app/assets/stylesheets/buttons.css b/app/assets/stylesheets/buttons.css
index 11e52668c..e2710006d 100644
--- a/app/assets/stylesheets/buttons.css
+++ b/app/assets/stylesheets/buttons.css
@@ -39,6 +39,31 @@
       opacity: 0.3;
       pointer-events: none;
     }
+
+    [aria-busy] &:disabled {
+      position: relative;
+
+      > * {
+        visibility: hidden;
+      }
+
+      &::after {
+        --mask: no-repeat radial-gradient(#000 68%,#0000 71%);
+        --size: 1.25em;
+
+        -webkit-mask: var(--mask), var(--mask), var(--mask);
+        -webkit-mask-size: 28% 45%;
+        animation: submitting 1s infinite linear;
+        aspect-ratio: 8/5;
+        background: currentColor;
+        content: "";
+        inline-size: var(--size);
+        inset: 50%;
+        margin-block: calc((var(--size) / 3) * -1);
+        margin-inline: calc((var(--size) / 2) * -1);
+        position: absolute;
+      }
+    }
   }
 
   /* Variants

From 48fd1f3502bfde52854de7a2f282b64312d06af5 Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Thu, 30 Oct 2025 12:16:35 -0500
Subject: [PATCH 08/12] Use debounced submit to prevent duplicate submissions

---
 app/views/cards/comments/_new.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/cards/comments/_new.html.erb b/app/views/cards/comments/_new.html.erb
index 015c93a83..e2b51ffe3 100644
--- a/app/views/cards/comments/_new.html.erb
+++ b/app/views/cards/comments/_new.html.erb
@@ -8,7 +8,7 @@
       <%= form_with model: Comment.new, url: card_comments_path(card), class: "flex flex-column gap full-width",
             data: { controller: "form local-save",
                   local_save_key_value: "comment-#{card.id}",
-                  action: "turbo:submit-end->local-save#submit keydown.ctrl+enter->form#submit:prevent keydown.meta+enter->form#submit:prevent keydown.esc->form#cancel:stop" } do |form| %>
+                  action: "turbo:submit-end->local-save#submit keydown.ctrl+enter->form#debouncedSubmit:prevent keydown.meta+enter->form#debouncedSubmit:prevent keydown.esc->form#cancel:stop" } do |form| %>
         <%= form.rich_textarea :body, required: true, placeholder: new_comment_placeholder(card),
               data: { local_save_target: "input", action: "lexxy:change->form#disableSubmitWhenInvalid lexxy:change->local-save#save turbo:morph-element->local-save#restoreContent" } do %>
           <%= general_prompts(@card.collection) %>

From 2c04199277d4a2e51319635af21fd2323c619c8d Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Thu, 30 Oct 2025 12:36:15 -0500
Subject: [PATCH 09/12] No longer used

And hilariously was unintentionally renamed when we went from 'pop' to
'closure'
---
 .../controllers/closureover_controller.js     | 43 -------------------
 1 file changed, 43 deletions(-)
 delete mode 100644 app/javascript/controllers/closureover_controller.js

diff --git a/app/javascript/controllers/closureover_controller.js b/app/javascript/controllers/closureover_controller.js
deleted file mode 100644
index 3c872f071..000000000
--- a/app/javascript/controllers/closureover_controller.js
+++ /dev/null
@@ -1,43 +0,0 @@
-import { Controller } from "@hotwired/stimulus"
-
-const BOTTOM_THRESHOLD = 0
-
-export default class extends Controller {
-  static targets = [ "menu" ]
-  static classes = [ "orientationTop" ]
-
-  close() {
-    this.menuTarget.close()
-    this.#orient()
-  }
-
-  open() {
-    this.menuTarget.show()
-    this.#orient()
-  }
-
-  toggle() {
-    this.menuTarget.open ? this.close() : this.open()
-  }
-
-  closeOnClickOutside({ target }) {
-    if (!this.element.contains(target)) this.close()
-  }
-
-  #orient() {
-    this.element.classList.toggle(this.orientationTopClass, this.#distanceToBottom < BOTTOM_THRESHOLD)
-    this.menuTarget.style.setProperty("--max-width", this.#maxWidth + "px")
-  }
-
-  get #distanceToBottom() {
-    return window.innerHeight - this.#boundingClientRect.bottom
-  }
-
-  get #maxWidth() {
-    return window.innerWidth - this.#boundingClientRect.left
-  }
-
-  get #boundingClientRect() {
-    return this.menuTarget.getBoundingClientRect()
-  }
-}

From 4c7578c78277c4d17d668d9a340aa715c89e2700 Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Thu, 30 Oct 2025 13:50:21 -0500
Subject: [PATCH 10/12] Need to reset both inline values or the tooltip gets
 squished

---
 app/assets/stylesheets/tooltips.css | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/assets/stylesheets/tooltips.css b/app/assets/stylesheets/tooltips.css
index 9a31f48d8..20a02985b 100644
--- a/app/assets/stylesheets/tooltips.css
+++ b/app/assets/stylesheets/tooltips.css
@@ -19,12 +19,12 @@
       text-overflow: ellipsis;
 
       &.orient-right {
-        inset-inline-start: 0;
+        inset-inline: 0 auto;
         translate: 0 -100%;
       }
 
       &.orient-left {
-        inset-inline-end: 0;
+        inset-inline: auto 0;
         translate: 0 -100%;
       }
     }

From ec620f98a6e494713ebea7aeff9b2e567839b69b Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Thu, 30 Oct 2025 13:57:20 -0500
Subject: [PATCH 11/12] Need to opt-out of orient-top if there isn't space
 above

---
 app/assets/stylesheets/popup.css              | 4 ++++
 app/javascript/helpers/orientation_helpers.js | 7 ++++++-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/app/assets/stylesheets/popup.css b/app/assets/stylesheets/popup.css
index b258086d0..fbfd8f7c2 100644
--- a/app/assets/stylesheets/popup.css
+++ b/app/assets/stylesheets/popup.css
@@ -32,6 +32,10 @@
 
     &.orient-top {
       inset-block: auto 0;
+
+      &.orient-bottom {
+        inset-block: 0 auto;
+      }
     }
 
     &:has(.popup__footer) {
diff --git a/app/javascript/helpers/orientation_helpers.js b/app/javascript/helpers/orientation_helpers.js
index 4e3f1a3a5..672275778 100644
--- a/app/javascript/helpers/orientation_helpers.js
+++ b/app/javascript/helpers/orientation_helpers.js
@@ -4,7 +4,8 @@ export function orient(el, orient = true) {
   const directions = [
     ["orient-left", spaceOnRight],
     ["orient-right", spaceOnLeft],
-    ["orient-top", spaceOnBottom]
+    ["orient-top", spaceOnBottom],
+    ["orient-bottom", spaceOnTop]
   ];
 
   directions.forEach(([className, fn]) =>
@@ -23,3 +24,7 @@ function spaceOnRight(el) {
 function spaceOnBottom(el) {
   return window.innerHeight - el.getBoundingClientRect().bottom
 }
+
+function spaceOnTop(el) {
+  return el.getBoundingClientRect().top
+}

From 4ecf81cd3247bdc3155720de46b7c72c23cf25cf Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Thu, 30 Oct 2025 14:00:56 -0500
Subject: [PATCH 12/12] Resize dialogs to fit viewport

---
 app/assets/stylesheets/popup.css                |  1 -
 app/javascript/controllers/dialog_controller.js |  4 ++++
 app/javascript/helpers/sizing_helpers.js        | 12 ++++++++++++
 3 files changed, 16 insertions(+), 1 deletion(-)
 create mode 100644 app/javascript/helpers/sizing_helpers.js

diff --git a/app/assets/stylesheets/popup.css b/app/assets/stylesheets/popup.css
index fbfd8f7c2..852787701 100644
--- a/app/assets/stylesheets/popup.css
+++ b/app/assets/stylesheets/popup.css
@@ -8,7 +8,6 @@
     --popup-item-padding-inline: 0.5rem;
 
     inset: 0 auto auto 50%;
-    max-block-size: 80vh;
     min-inline-size: min(25ch, calc(100dvw - var(--block-space-double)));
     max-inline-size: min(55ch, calc(100dvw - var(--block-space-double)));
     overflow: auto;
diff --git a/app/javascript/controllers/dialog_controller.js b/app/javascript/controllers/dialog_controller.js
index e9f8f990c..16204bd28 100644
--- a/app/javascript/controllers/dialog_controller.js
+++ b/app/javascript/controllers/dialog_controller.js
@@ -1,5 +1,6 @@
 import { Controller } from "@hotwired/stimulus"
 import { orient } from "helpers/orientation_helpers"
+import { limitHeightToViewport } from "helpers/sizing_helpers"
 
 export default class extends Controller {
   static targets = [ "dialog" ]
@@ -21,6 +22,8 @@ export default class extends Controller {
       orient(this.dialogTarget)
     }
 
+    limitHeightToViewport(this.dialogTarget, true)
+
     this.dialogTarget.setAttribute("aria-hidden", "false")
     this.dispatch("show")
   }
@@ -38,6 +41,7 @@ export default class extends Controller {
     this.dialogTarget.setAttribute("aria-hidden", "true")
     this.dialogTarget.blur()
     orient(this.dialogTarget, false)
+    limitHeightToViewport(this.dialogTarget, false)
   }
 
   closeOnClickOutside({ target }) {
diff --git a/app/javascript/helpers/sizing_helpers.js b/app/javascript/helpers/sizing_helpers.js
new file mode 100644
index 000000000..4b7e99645
--- /dev/null
+++ b/app/javascript/helpers/sizing_helpers.js
@@ -0,0 +1,12 @@
+export function limitHeightToViewport(el, sizing = true, margin = 64) {
+  if (!el) return
+
+  if (sizing) {
+    const rect = el.getBoundingClientRect()
+    const top = Math.max(rect.top, margin)
+    const max = Math.max(0, window.innerHeight - margin - top)
+    el.style.maxHeight = `${max}px`
+  } else {
+    el.style.maxHeight = ""
+  }
+}
\ No newline at end of file