From 23f1ba1b3f0279e1d3faf46c69416e1883d76471 Mon Sep 17 00:00:00 2001
From: Ryan Hughes <ryan@heyoodle.com>
Date: Mon, 18 May 2026 01:17:35 -0400
Subject: [PATCH] Dropdown/SearchableDropdown: 4px spacing between popup rows

---
 default/quickshell/omarchy-shell/Ui/Dropdown.qml           | 5 +++--
 default/quickshell/omarchy-shell/Ui/SearchableDropdown.qml | 5 +++--
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/default/quickshell/omarchy-shell/Ui/Dropdown.qml b/default/quickshell/omarchy-shell/Ui/Dropdown.qml
index 9187b8f7..00817a13 100644
--- a/default/quickshell/omarchy-shell/Ui/Dropdown.qml
+++ b/default/quickshell/omarchy-shell/Ui/Dropdown.qml
@@ -124,8 +124,8 @@ Item {
         x: 0
         y: trigger.height + 2
         width: trigger.width
-        implicitHeight: Math.min(root.options.length * root.popupRowHeight + 2,
-                                 root.popupRowHeight * 8 + 2)
+        implicitHeight: Math.min(root.options.length * root.popupRowHeight + Math.max(0, root.options.length - 1) * 4 + 2,
+                                 root.popupRowHeight * 8 + 7 * 4 + 2)
         padding: 1
         focus: true
 
@@ -143,6 +143,7 @@ Item {
 
         contentItem: ListView {
           id: optionList
+          spacing: 4
 
           Keys.onPressed: function(event) {
             if (event.key === Qt.Key_Escape) { popup.close(); event.accepted = true }
diff --git a/default/quickshell/omarchy-shell/Ui/SearchableDropdown.qml b/default/quickshell/omarchy-shell/Ui/SearchableDropdown.qml
index b81695fe..d3a1b6ae 100644
--- a/default/quickshell/omarchy-shell/Ui/SearchableDropdown.qml
+++ b/default/quickshell/omarchy-shell/Ui/SearchableDropdown.qml
@@ -146,8 +146,8 @@ Item {
         y: trigger.height + 2
         width: trigger.width
         implicitHeight: Math.max(root.popupMinHeight,
-                                 Math.min(root.filtered.length * root.popupRowHeight + 50,
-                                          root.popupRowHeight * 6 + 50))
+                                 Math.min(root.filtered.length * root.popupRowHeight + Math.max(0, root.filtered.length - 1) * 4 + 50,
+                                          root.popupRowHeight * 6 + 5 * 4 + 50))
         padding: 1
         focus: true
 
@@ -229,6 +229,7 @@ Item {
             ListView {
               id: resultList
               anchors.fill: parent
+              spacing: 4
               clip: true
               boundsBehavior: Flickable.StopAtBounds
               model: root.filtered