From 5768eb26e8024db373d466e84f136a6f94915bad Mon Sep 17 00:00:00 2001
From: Andy Smith <andy@37signals.com>
Date: Mon, 29 Sep 2025 15:52:02 -0500
Subject: [PATCH] Better padding for popup items

---
 app/assets/stylesheets/popup.css | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/app/assets/stylesheets/popup.css b/app/assets/stylesheets/popup.css
index c864655a4..11c394ff8 100644
--- a/app/assets/stylesheets/popup.css
+++ b/app/assets/stylesheets/popup.css
@@ -5,7 +5,7 @@
     --panel-padding: var(--block-space);
     --panel-size: auto;
     --popup-icon-size: 24px;
-    --popup-item-padding-inline: var(--inline-space-half);
+    --popup-item-padding-inline: 0.5rem;
 
     inset: 0 auto auto 50%;
     max-block-size: 80vh;
@@ -122,14 +122,6 @@
         }
       }
     }
-
-    &:has(> :first-child:not(.popup__btn)) {
-      padding-inline-start: var(--popup-item-padding-inline);
-    }
-
-    &:has(> :last-child:not(.popup__btn)) {
-      padding-inline-end: var(--popup-item-padding-inline);
-    }
   }
 
   /* The actionable thing with padding within popup__item */
@@ -165,6 +157,7 @@
     --icon-size: 1em;
 
     inline-size: var(--popup-icon-size);
+    margin-inline-start: var(--popup-item-padding-inline);
   }
 
   .popup__radio {
@@ -172,6 +165,7 @@
 
     block-size: var(--popup-icon-size);
     inline-size: var(--popup-icon-size);
+    margin-inline-start: var(--popup-item-padding-inline);
     flex-shrink: 0;
 
     &:hover {