From 077aeeeca8d8e6f17723af77656c364f9e999727 Mon Sep 17 00:00:00 2001
From: uncenter <uncenter@uncenter.dev>
Date: Sat, 12 Jul 2025 13:11:28 -0400
Subject: [PATCH] appview/pages: show user avatar in profile dropdown

Renames user/fragments/picHandle to user/fragments/picHandleLink, since it now wraps a simpler user/fragments/picHandle that doesn't include the link. The topbar should now display the avatar next to the handle in the same style, without the link as it was before.
---
 appview/pages/templates/layouts/topbar.html               | 5 +++--
 appview/pages/templates/repo/issues/issue.html            | 2 +-
 appview/pages/templates/repo/issues/issues.html           | 2 +-
 .../pages/templates/repo/pulls/fragments/pullHeader.html  | 2 +-
 appview/pages/templates/repo/pulls/pulls.html             | 2 +-
 appview/pages/templates/timeline.html                     | 8 ++++----
 appview/pages/templates/user/fragments/picHandle.html     | 4 ++--
 appview/pages/templates/user/fragments/picHandleLink.html | 5 +++++
 8 files changed, 18 insertions(+), 12 deletions(-)
 create mode 100644 appview/pages/templates/user/fragments/picHandleLink.html

diff --git a/appview/pages/templates/layouts/topbar.html b/appview/pages/templates/layouts/topbar.html
index 50d9ed9..0851187 100644
--- a/appview/pages/templates/layouts/topbar.html
+++ b/appview/pages/templates/layouts/topbar.html
@@ -38,12 +38,13 @@
     <summary
         class="cursor-pointer list-none"
     >
-        {{ didOrHandle .Did .Handle }}
+        {{ $user := didOrHandle .Did .Handle }}
+        {{ template "user/fragments/picHandle" $user }}
     </summary>
     <div
         class="absolute flex flex-col right-0 mt-4 p-4 rounded w-48 bg-white dark:bg-gray-800 dark:text-white border border-gray-200 dark:border-gray-700"
     >
-        <a href="/{{ didOrHandle .Did .Handle }}">profile</a>
+        <a href="/{{ $user }}">profile</a>
         <a href="/knots">knots</a>
         <a href="/spindles">spindles</a>
         <a href="/settings">settings</a>
diff --git a/appview/pages/templates/repo/issues/issue.html b/appview/pages/templates/repo/issues/issue.html
index f26a536..f4035c2 100644
--- a/appview/pages/templates/repo/issues/issue.html
+++ b/appview/pages/templates/repo/issues/issue.html
@@ -33,7 +33,7 @@
             <span class="text-gray-500 dark:text-gray-400 text-sm flex flex-wrap items-center gap-1">
                 opened by
                 {{ $owner := didOrHandle .Issue.OwnerDid .IssueOwnerHandle }}
-                {{ template "user/fragments/picHandle" $owner }}
+                {{ template "user/fragments/picHandleLink" $owner }}
                <span class="select-none before:content-['\00B7']"></span>
                 <time title="{{ .Issue.Created | longTimeFmt }}">
                     {{ .Issue.Created | timeFmt }}
diff --git a/appview/pages/templates/repo/issues/issues.html b/appview/pages/templates/repo/issues/issues.html
index 1501159..8087888 100644
--- a/appview/pages/templates/repo/issues/issues.html
+++ b/appview/pages/templates/repo/issues/issues.html
@@ -66,7 +66,7 @@
 
       <span class="ml-1">
           {{ $owner := index $.DidHandleMap .OwnerDid }}
-          {{ template "user/fragments/picHandle" $owner }}
+          {{ template "user/fragments/picHandleLink" $owner }}
       </span>
 
       <span class="before:content-['·']">
diff --git a/appview/pages/templates/repo/pulls/fragments/pullHeader.html b/appview/pages/templates/repo/pulls/fragments/pullHeader.html
index 24cb25a..ec341c8 100644
--- a/appview/pages/templates/repo/pulls/fragments/pullHeader.html
+++ b/appview/pages/templates/repo/pulls/fragments/pullHeader.html
@@ -29,7 +29,7 @@
         <span class="text-gray-500 dark:text-gray-400 text-sm flex flex-wrap items-center gap-1">
             opened by
             {{ $owner := index $.DidHandleMap .Pull.OwnerDid }}
-            {{ template "user/fragments/picHandle" $owner }}
+            {{ template "user/fragments/picHandleLink" $owner }}
             <span class="select-none before:content-['\00B7']"></span>
             <time>{{ .Pull.Created | timeFmt }}</time>
 
diff --git a/appview/pages/templates/repo/pulls/pulls.html b/appview/pages/templates/repo/pulls/pulls.html
index 2c46070..95fe805 100644
--- a/appview/pages/templates/repo/pulls/pulls.html
+++ b/appview/pages/templates/repo/pulls/pulls.html
@@ -76,7 +76,7 @@
                     </span>
 
                     <span class="ml-1">
-                        {{ template "user/fragments/picHandle" $owner }}
+                        {{ template "user/fragments/picHandleLink" $owner }}
                     </span>
 
                     <span>
diff --git a/appview/pages/templates/timeline.html b/appview/pages/templates/timeline.html
index 02cba48..f6e9d35 100644
--- a/appview/pages/templates/timeline.html
+++ b/appview/pages/templates/timeline.html
@@ -61,7 +61,7 @@
                         {{ $userHandle :=  index $.DidHandleMap .Repo.Did }}
                         <div class="flex items-center">
                             <p class="text-gray-600 dark:text-gray-300 flex flex-wrap items-center gap-2">
-                                {{ template "user/fragments/picHandle" $userHandle }}
+                                {{ template "user/fragments/picHandleLink" $userHandle }}
                                 {{ if .Source }}
                                     forked
                                     <a
@@ -95,9 +95,9 @@
                         {{ $subjectHandle :=  index $.DidHandleMap .Follow.SubjectDid }}
                         <div class="flex items-center">
                             <p class="text-gray-600 dark:text-gray-300 flex flex-wrap items-center gap-2">
-                                {{ template "user/fragments/picHandle" $userHandle }}
+                                {{ template "user/fragments/picHandleLink" $userHandle }}
                                 followed
-                                {{ template "user/fragments/picHandle" $subjectHandle }}
+                                {{ template "user/fragments/picHandleLink" $subjectHandle }}
                                 <time
                                     class="text-gray-700 dark:text-gray-400 text-xs"
                                     >{{ .Follow.FollowedAt | timeFmt }}</time
@@ -109,7 +109,7 @@
                         {{ $repoOwnerHandle :=  index $.DidHandleMap .Star.Repo.Did }}
                         <div class="flex items-center">
                             <p class="text-gray-600 dark:text-gray-300 flex items-center gap-2">
-                                {{ template "user/fragments/picHandle" $starrerHandle }}
+                                {{ template "user/fragments/picHandleLink" $starrerHandle }}
                                 starred
                                 <a
                                     href="/{{ $repoOwnerHandle }}/{{ .Star.Repo.Name }}"
diff --git a/appview/pages/templates/user/fragments/picHandle.html b/appview/pages/templates/user/fragments/picHandle.html
index a1c1085..c2bd2e6 100644
--- a/appview/pages/templates/user/fragments/picHandle.html
+++ b/appview/pages/templates/user/fragments/picHandle.html
@@ -1,10 +1,10 @@
 {{ define "user/fragments/picHandle" }}
-<a href="/{{ . }}" class="flex items-center">
+<div class="flex items-center">
     <img
         src="{{ tinyAvatar . }}"
         alt="{{ . }}"
         class="rounded-full h-6 w-6 mr-1 border border-gray-300 dark:border-gray-700"
     />
     {{ . | truncateAt30 }}
-</a>
+</div>
 {{ end }}
diff --git a/appview/pages/templates/user/fragments/picHandleLink.html b/appview/pages/templates/user/fragments/picHandleLink.html
new file mode 100644
index 0000000..1eb8ff9
--- /dev/null
+++ b/appview/pages/templates/user/fragments/picHandleLink.html
@@ -0,0 +1,5 @@
+{{ define "user/fragments/picHandleLink" }}
+<a href="/{{ . }}">
+    {{ template "user/fragments/picHandle" . }}
+</a>
+{{ end }}
-- 
2.43.0