From 26aef50369db0d953fb01f532aae48cb615e6878 Mon Sep 17 00:00:00 2001
From: oppiliappan <me@oppi.li>
Date: Mon, 11 May 2026 14:46:03 +0100
Subject: [PATCH] appview/pages: assortment of UI fixes

- fork button
- fork pages
- text on star/fork buttons in mobile/desktop

Signed-off-by: oppiliappan <me@oppi.li>
---
 .../pages/templates/fragments/starBtn.html    |  2 +-
 appview/pages/templates/layouts/repobase.html |  2 +-
 appview/pages/templates/repo/forks.html       | 25 ++++++++++---------
 appview/repo/repo.go                          |  5 +++-
 4 files changed, 19 insertions(+), 15 deletions(-)

diff --git a/appview/pages/templates/fragments/starBtn.html b/appview/pages/templates/fragments/starBtn.html
index 91a56d03..2f2998e8 100644
--- a/appview/pages/templates/fragments/starBtn.html
+++ b/appview/pages/templates/fragments/starBtn.html
@@ -22,7 +22,7 @@
                 {{ i "star" "w-4 h-4 inline group-[.htmx-request]:hidden" }}
             {{ end }}
             {{ i "loader-circle" "w-4 h-4 animate-spin hidden group-[.htmx-request]:inline" }}
-            <span class="text-sm md:hidden group-[.htmx-request]:hidden">{{ if .IsStarred }}unstar{{ else }}star{{ end }}</span>
+            <span class="text-sm group-[.htmx-request]:hidden">{{ if .IsStarred }}unstar{{ else }}star{{ end }}</span>
         </button>
         {{ if .RepoName }}
             {{ $did := .SubjectAt.Authority | string }}
diff --git a/appview/pages/templates/layouts/repobase.html b/appview/pages/templates/layouts/repobase.html
index 557df94d..ecf095fb 100644
--- a/appview/pages/templates/layouts/repobase.html
+++ b/appview/pages/templates/layouts/repobase.html
@@ -119,7 +119,7 @@
             "RepoName" .RepoInfo.Name) }}
       <div class="flex w-full min-h-[30px] items-stretch overflow-hidden rounded border border-gray-200 dark:border-gray-700 bg-white dark:bg-gray-800 shadow-sm">
         <a
-          class="btn text-sm no-underline hover:no-underline flex items-center gap-2 group"
+          class="flex flex-1 justify-center gap-2 items-center px-2 text-sm no-underline hover:no-underline group hover:bg-gray-50 dark:hover:bg-gray-700"
           hx-boost="true"
           href="/{{ .RepoInfo.FullName }}/fork"
         >
diff --git a/appview/pages/templates/repo/forks.html b/appview/pages/templates/repo/forks.html
index 5286db78..c105222e 100644
--- a/appview/pages/templates/repo/forks.html
+++ b/appview/pages/templates/repo/forks.html
@@ -4,19 +4,11 @@
     <h2 class="text-sm uppercase font-bold">Forked by</h2>
     <div class="grid grid-cols-1 sm:grid-cols-2 lg:grid-cols-3 gap-4">
       {{ range .Forks }}
-        {{ $handle := resolve .Did }}
-        {{ $name := .Name}}
         <div class="border border-gray-200 dark:border-gray-700 rounded p-4">
-          <div class="flex items-center gap-3">
-            {{ template "user/fragments/picLink" (list .Did "size-10") }}
-            <div class="flex-1 min-w-0">
-              <a href="/{{ $handle }}" class="block truncate">{{ $handle }}</a>
-              <a href="/{{ $handle }}/{{ $name }}" class="block truncate">{{ $name }}</a>
-              <p class="text-sm text-gray-500 dark:text-gray-400">
-                forked {{ .Created | relTimeFmt }}
-              </p>
-            </div>
-          </div>
+          {{ template "forkOwnerAndName" . }}
+          <p class="text-sm text-gray-500 dark:text-gray-400">
+            forked {{ .Created | relTimeFmt }}
+          </p>
         </div>
       {{ end }}
       {{ if eq .TotalCount 0 }}
@@ -33,3 +25,12 @@
     {{ end }}
   </div>
 {{ end }}
+
+{{ define "forkOwnerAndName" }}
+  <div class="flex items-center gap-1 flex-wrap text-lg">
+    {{ template "user/fragments/picHandleLink" .Did }}
+    <span class="select-none">/</span>
+    {{ $repoOwner := resolve .Did }}
+    <a href="/$repoOwner/{{ .Name }}">{{ .Name }}</a>
+  </div>
+{{ end }}
diff --git a/appview/repo/repo.go b/appview/repo/repo.go
index c05b5b56..aa3af6d3 100644
--- a/appview/repo/repo.go
+++ b/appview/repo/repo.go
@@ -1525,13 +1525,16 @@ func (rp *Repo) Forks(w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
-	rp.pages.RepoForks(w, pages.RepoForksParams{
+	err = rp.pages.RepoForks(w, pages.RepoForksParams{
 		LoggedInUser: user,
 		RepoInfo:     rp.repoResolver.GetRepoInfo(r, user),
 		Forks:        forks,
 		Page:         page,
 		TotalCount:   totalCount,
 	})
+	if err != nil {
+		l.Error("failed to render page", "err", err)
+	}
 }
 
 // this is used to rollback changes made to the PDS
-- 
2.51.2