Merge pull request #1431 from basecamp/jz/10-30-25-pm

JZ 10/30/25 pm

app/assets/stylesheets/popup.css

@@ -29,14 +29,6 @@
       transform: translateX(0);
     }
 
-    &.orient-top {
-      inset-block: auto 0;
-
-      &.orient-bottom {
-        inset-block: 0 auto;
-      }
-    }
-
     &:has(.popup__footer) {
       --panel-padding: var(--block-space) var(--block-space) 0 var(--block-space);
     }

app/javascript/helpers/orientation_helpers.js

@@ -4,8 +4,7 @@ export function orient(el, orient = true) {
   const directions = [
     ["orient-left", spaceOnRight],
     ["orient-right", spaceOnLeft],
-    ["orient-top", spaceOnBottom],
-    ["orient-bottom", spaceOnTop]
+    ["orient-top", spaceOnBottom]
   ];
 
   directions.forEach(([className, fn]) =>
@@ -24,7 +23,3 @@ function spaceOnRight(el) {
 function spaceOnBottom(el) {
   return window.innerHeight - el.getBoundingClientRect().bottom
 }
-
-function spaceOnTop(el) {
-  return el.getBoundingClientRect().top
-}

app/views/bar/_bar.html.erb

@@ -1,4 +1,4 @@
-<div class="bar full-width" data-controller="bar" data-bar-dialog-outlet="#bar-model"
+<div class="bar full-width" data-controller="bar" data-bar-dialog-outlet="#bar-modal"
     data-bar-search-url-value="<%= search_path %>">
   <div class="flex justify-center bar__placeholder" data-bar-target="buttonsContainer">
     <%= tag.button class: "btn btn--plain", data: {