From ae18c23a0082796f44834cae2e10667e748bc8ae Mon Sep 17 00:00:00 2001
From: oppiliappan <me@oppi.li>
Date: Tue, 20 Jan 2026 11:49:33 +0000
Subject: [PATCH] appview/pages: finesse pull review page styles

- transparent blue caused text ovelaps
- mobile view hide the split/unified buttons
- transparent topbar caused text overlaps

Signed-off-by: oppiliappan <me@oppi.li>
---
 appview/pages/templates/repo/fragments/diff.html |  8 +++-----
 appview/pages/templates/repo/pulls/pull.html     | 12 ++++++++++--
 2 files changed, 13 insertions(+), 7 deletions(-)

diff --git a/appview/pages/templates/repo/fragments/diff.html b/appview/pages/templates/repo/fragments/diff.html
index d6836235..af614302 100644
--- a/appview/pages/templates/repo/fragments/diff.html
+++ b/appview/pages/templates/repo/fragments/diff.html
@@ -23,7 +23,7 @@
   {{ block "subsCheckbox" $ }} {{ end }}
 
   <!-- top bar -->
-  <div class="sticky top-0 z-30 flex items-center gap-2 col-span-full h-12 p-2 {{ if $root }}mt-4{{ end }}">
+  <div class="sticky top-0 z-30 bg-slate-100 dark:bg-gray-900 flex items-center gap-2 col-span-full h-12 p-2 {{ if $root }}mt-4{{ end }}">
     <!-- left panel toggle -->
     {{ template "filesToggle" . }}
 
@@ -31,7 +31,7 @@
     {{ $stat := $diff.Stats }}
     {{ $count := len $diff.ChangedFiles }}
     {{ template "repo/fragments/diffStatPill" $stat }}
-    <span class="text-xs text-gray-600 dark:text-gray-400">{{ $count }} changed file{{ if ne $count 1 }}s{{ end }}</span>
+    <span class="text-xs text-gray-600 dark:text-gray-400 hidden md:inline-flex">{{ $count }} changed file{{ if ne $count 1 }}s{{ end }}</span>
 
     {{ if $root }}
       {{ if $root.IsInterdiff }}
@@ -70,9 +70,7 @@
     {{ template "collapseToggle" }}
 
     <!-- diff options -->
-    <div class="hidden md:block">
-      {{ template "repo/fragments/diffOpts" $opts }}
-    </div>
+    {{ template "repo/fragments/diffOpts" $opts }}
 
     <!-- right panel toggle -->
     {{ block "subsToggle" $ }} {{ end }}
diff --git a/appview/pages/templates/repo/pulls/pull.html b/appview/pages/templates/repo/pulls/pull.html
index f3993b9e..08ea9b32 100644
--- a/appview/pages/templates/repo/pulls/pull.html
+++ b/appview/pages/templates/repo/pulls/pull.html
@@ -246,7 +246,15 @@
   {{ $lastIdx := index . 2 }}
   {{ $root := index . 3 }}
   {{ $round := $item.RoundNumber }}
-  <div class="{{ if eq $round 0 }}rounded-b{{ else }}rounded{{ end }} px-6 py-4 pr-2 pt-2 {{ if eq $root.ActiveRound $round }}bg-blue-100 dark:bg-blue-900/50 border-b border-blue-200 dark:border-blue-700{{ else }}bg-white dark:bg-gray-800 border-b border-gray-200 dark:border-gray-700{{ end }} flex gap-2 sticky top-0 z-20">
+  <div class="
+    {{ if eq $round 0 }}rounded-b{{ else }}rounded{{ end }}
+    px-6 py-4 pr-2 pt-2
+    {{ if eq $root.ActiveRound $round }}
+      bg-blue-100 dark:bg-blue-900 border-b border-blue-200 dark:border-blue-700
+    {{ else }}
+      bg-white dark:bg-gray-800 border-b border-gray-200 dark:border-gray-700
+    {{ end }}
+    flex gap-2 sticky top-0 z-20">
     <!-- left column: just profile picture -->
     <div class="flex-shrink-0 pt-2">
       <img
@@ -286,7 +294,7 @@
     <div class="flex gap-2 items-center">
       {{ if ne $root.ActiveRound $round }}
       <a class="btn-flat flex items-center gap-2 no-underline hover:no-underline text-sm"
-        href="/{{ $root.RepoInfo.FullName }}/pulls/{{ $root.Pull.PullId }}/round/{{ $round }}?{{ safeUrl $root.DiffOpts.Encode }}">
+        href="/{{ $root.RepoInfo.FullName }}/pulls/{{ $root.Pull.PullId }}/round/{{ $round }}?{{ safeUrl $root.DiffOpts.Encode }}#round-#{{ $round }}">
         {{ i "diff" "w-4 h-4" }}
         diff
       </a>
-- 
2.51.2