From b17662e6db92e464a612c8757fa1791a08a79fee Mon Sep 17 00:00:00 2001
From: BrookJeynes <jeynesbrook@gmail.com>
Date: Sat, 10 May 2025 17:42:54 +1000
Subject: [PATCH] appview: repo/pulls: change open/merged/closed pulls select
 to buttons

---
 .../pages/templates/repo/issues/issues.html   |  4 +-
 appview/pages/templates/repo/pulls/pulls.html | 39 +++++++++++--------
 2 files changed, 24 insertions(+), 19 deletions(-)

diff --git a/appview/pages/templates/repo/issues/issues.html b/appview/pages/templates/repo/issues/issues.html
index 3b3cdb5a..3d816924 100644
--- a/appview/pages/templates/repo/issues/issues.html
+++ b/appview/pages/templates/repo/issues/issues.html
@@ -5,14 +5,14 @@
   <div class="flex gap-4">
     <a
         href="?state=open"
-        class="flex items-center gap-2 {{ if .FilteringByOpen }}font-bold{{ end }}"
+        class="flex items-center gap-2 {{ if .FilteringByOpen }}font-bold {{ else }}text-gray-500 dark:text-gray-400{{ end }}"
         >
         {{ i "circle-dot" "w-4 h-4" }}
         <span>{{ .RepoInfo.Stats.IssueCount.Open }} open</span>
     </a>
     <a
         href="?state=closed"
-        class="flex items-center gap-2 {{ if not .FilteringByOpen }}font-bold{{ end }}"
+        class="flex items-center gap-2 {{ if not .FilteringByOpen }}font-bold {{ else }}text-gray-500 dark:text-gray-400{{ end }}"
         >
         {{ i "ban" "w-4 h-4" }}
         <span>{{ .RepoInfo.Stats.IssueCount.Closed }} closed</span>
diff --git a/appview/pages/templates/repo/pulls/pulls.html b/appview/pages/templates/repo/pulls/pulls.html
index 6db62e17..0d09c8aa 100644
--- a/appview/pages/templates/repo/pulls/pulls.html
+++ b/appview/pages/templates/repo/pulls/pulls.html
@@ -2,24 +2,29 @@
 
 {{ define "repoContent" }}
     <div class="flex justify-between items-center">
-        <p class="dark:text-white">
-            filtering
-            <select
-                class="border p-1 bg-white dark:bg-gray-800 border-gray-200 dark:border-gray-600 dark:text-white"
-                onchange="window.location.href = '/{{ .RepoInfo.FullName }}/pulls?state=' + this.value"
+        <div class="flex gap-4">
+            <a
+                href="?state=open"
+                class="flex items-center gap-2 {{ if .FilteringBy.IsOpen }}font-bold {{ else }}text-gray-500 dark:text-gray-400{{ end }}"
             >
-                <option value="open" {{ if .FilteringBy.IsOpen }}selected{{ end }}>
-                    open ({{ .RepoInfo.Stats.PullCount.Open }})
-                </option>
-                <option value="merged" {{ if .FilteringBy.IsMerged }}selected{{ end }}>
-                    merged ({{ .RepoInfo.Stats.PullCount.Merged }})
-                </option>
-                <option value="closed" {{ if .FilteringBy.IsClosed }}selected{{ end }}>
-                    closed ({{ .RepoInfo.Stats.PullCount.Closed }})
-                </option>
-            </select>
-            pull requests
-        </p>
+                {{ i "circle-dot" "w-4 h-4" }}
+                <span>{{ .RepoInfo.Stats.PullCount.Open }} open</span>
+            </a>
+            <a
+                href="?state=merged"
+                class="flex items-center gap-2 {{ if .FilteringBy.IsMerged }}font-bold {{ else }}text-gray-500 dark:text-gray-400{{ end }}"
+            >
+                {{ i "git-merge" "w-4 h-4" }}
+                <span>{{ .RepoInfo.Stats.PullCount.Merged }} merged</span>
+            </a>
+            <a
+                href="?state=closed"
+                class="flex items-center gap-2 {{ if .FilteringBy.IsClosed }}font-bold {{ else }}text-gray-500 dark:text-gray-400{{ end }}"
+            >
+                {{ i "ban" "w-4 h-4" }}
+                <span>{{ .RepoInfo.Stats.PullCount.Closed }} closed</span>
+            </a>
+        </div>
         <a
             href="/{{ .RepoInfo.FullName }}/pulls/new"
             class="btn text-sm flex items-center gap-2 no-underline hover:no-underline"
-- 
2.51.2