From ebaa856e0c0aa46f36a33af74b54ccba0718ca1a Mon Sep 17 00:00:00 2001
From: Caleb Allen <caleballen1276@gmail.com>
Date: Sat, 21 Mar 2026 13:46:26 -0400
Subject: [PATCH] fix: align combobox height to operator select via grid
 stretch

- Grid rows use align-items: stretch so all cells share the select's height
- Combobox host, wrapper, and input all set height: 100% to fill the cell
- Remove button uses align-self: center to stay centered instead of stretching
---
 frontend/src/components/combobox/combobox.ts                  | 4 +++-
 .../components/smart-playlist-editor/smart-playlist-editor.ts | 4 ++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/frontend/src/components/combobox/combobox.ts b/frontend/src/components/combobox/combobox.ts
index 0ce0b3b..db0544d 100644
--- a/frontend/src/components/combobox/combobox.ts
+++ b/frontend/src/components/combobox/combobox.ts
@@ -63,12 +63,14 @@ export class YjCombobox extends LitElement {
             :host {
                 display: inline-block;
                 width: 100%;
+                height: 100%;
             }
 
             .combobox-wrapper {
                 position: relative;
                 display: inline-block;
                 width: 100%;
+                height: 100%;
             }
 
             input {
@@ -80,8 +82,8 @@ export class YjCombobox extends LitElement {
                 font-size: var(--yj-text-md);
                 font-family: inherit;
                 width: 100%;
+                height: 100%;
                 box-sizing: border-box;
-                height: 26px;
             }
 
             input:focus {
diff --git a/frontend/src/components/smart-playlist-editor/smart-playlist-editor.ts b/frontend/src/components/smart-playlist-editor/smart-playlist-editor.ts
index d1c61ff..4b5a205 100644
--- a/frontend/src/components/smart-playlist-editor/smart-playlist-editor.ts
+++ b/frontend/src/components/smart-playlist-editor/smart-playlist-editor.ts
@@ -194,7 +194,7 @@ export class SmartPlaylistEditor extends LitElement {
                 display: grid;
                 grid-template-columns: 160px 140px 1fr 28px;
                 gap: 6px;
-                align-items: start;
+                align-items: stretch;
             }
 
             .rule-row.between-row {
@@ -239,6 +239,7 @@ export class SmartPlaylistEditor extends LitElement {
                 display: flex;
                 align-items: center;
                 justify-content: center;
+                align-self: center;
                 width: 24px;
                 height: 24px;
                 border: none;
@@ -248,7 +249,6 @@ export class SmartPlaylistEditor extends LitElement {
                 cursor: pointer;
                 font-size: 14px;
                 padding: 0;
-                margin-top: 2px;
                 transition: color 0.15s ease, background-color 0.15s ease;
             }