From 89b74b49694b62b1f2f49d2aae7218613890edef Mon Sep 17 00:00:00 2001
From: oppiliappan <me@oppi.li>
Date: Sun, 03 May 2026 15:08:11 +0000
Subject: [PATCH] appview/pages: switch drop-shadow to shadow in few places

drop-shadow seems to create a new stacking context and that messes with
z-index

Signed-off-by: oppiliappan <me@oppi.li>
---
 appview/pages/templates/repo/pulls/patch.html | 2 +-
 appview/pages/templates/repo/pulls/pull.html | 16 ++++++++--------
 appview/pages/templates/repo/issues/fragments/issueListing.html | 2 +-
 3 file(s) changed, 10 insertion(s)(+), 10 deletion(s)(-)

diff --git a/appview/pages/templates/repo/pulls/patch.html b/appview/pages/templates/repo/pulls/patch.html
--- a/appview/pages/templates/repo/pulls/patch.html
+++ b/appview/pages/templates/repo/pulls/patch.html
@@ -14,7 +14,7 @@
 {{ define "content" }}
 <section>
     <section
-        class="bg-white dark:bg-gray-800 p-6 rounded relative w-full mx-auto drop-shadow-sm dark:text-white"
+        class="bg-white dark:bg-gray-800 p-6 rounded relative w-full mx-auto shadow-sm dark:text-white"
     >
         <div class="flex gap-3 items-center mb-3">
             <a href="/{{ .RepoInfo.FullName }}/pulls/{{ .Pull.PullId }}/" class="flex items-center gap-2 font-medium">
diff --git a/appview/pages/templates/repo/pulls/pull.html b/appview/pages/templates/repo/pulls/pull.html
--- a/appview/pages/templates/repo/pulls/pull.html
+++ b/appview/pages/templates/repo/pulls/pull.html
@@ -136,7 +136,7 @@
   <div class="flex col-span-full">
     <!-- left panel -->
     <div id="files" class="w-0 hidden md:block overflow-hidden sticky top-12 max-h-screen overflow-y-auto pb-12">
-      <section class="overflow-x-auto text-sm px-6 py-2 border-b border-x border-gray-200 dark:border-gray-700 w-full mx-auto min-h-full rounded-b rounded-t-none bg-white dark:bg-gray-800 drop-shadow-sm">
+      <section class="overflow-x-auto text-sm px-6 py-2 border-b border-x border-gray-200 dark:border-gray-700 w-full mx-auto min-h-full rounded-b rounded-t-none bg-white dark:bg-gray-800 shadow-sm">
         {{ template "repo/fragments/fileTree" $diff.FileTree }}
       </section>
     </div>
@@ -172,14 +172,14 @@
   <div id="bottomSheetBackdrop" class="fixed inset-0 bg-black/50 md:hidden opacity-0 pointer-events-none transition-opacity duration-300 z-40"></div>
   <!-- right panel - bottom sheet on mobile, side panel on desktop -->
   <div id="subs" class="fixed bottom-0 left-0 right-0 z-50 w-full md:static md:z-auto md:max-h-screen md:sticky md:top-12 overflow-hidden">
-    <details open id="bottomSheet" class="rounded-t-2xl md:rounded-t drop-shadow-lg md:drop-shadow-none group/panel">
+    <details open id="bottomSheet" class="rounded-t-2xl md:rounded-t shadow-lg md:shadow-none group/panel">
       <summary class="
         flex gap-4 items-center justify-between
         rounded-t-2xl md:rounded-t cursor-pointer list-none p-4 md:h-12
         text-white md:text-black md:dark:text-white
         {{ $bgColor }}
         md:bg-white md:dark:bg-gray-800
-        drop-shadow-sm border-t md:border-x border-gray-200 dark:border-gray-700
+        shadow-sm border-t md:border-x border-gray-200 dark:border-gray-700
         md:pointer-events-none
         ">
         <h2 class="">History</h2>
@@ -488,7 +488,7 @@
 
 {{ define "mergeStatus" }}
   {{ if .Pull.State.IsClosed }}
-  <div class="bg-gray-50 dark:bg-gray-700 border border-black dark:border-gray-500 rounded drop-shadow-sm px-6 py-2 relative">
+  <div class="bg-gray-50 dark:bg-gray-700 border border-black dark:border-gray-500 rounded shadow-sm px-6 py-2 relative">
     <div class="flex items-center gap-2 text-black dark:text-white">
       {{ i "ban" "w-4 h-4" }}
       <span class="font-medium">closed without merging</span
@@ -496,7 +496,7 @@
     </div>
   </div>
   {{ else if .Pull.State.IsMerged }}
-  <div class="bg-purple-50 dark:bg-purple-900 border border-purple-500 rounded drop-shadow-sm px-6 py-2 relative">
+  <div class="bg-purple-50 dark:bg-purple-900 border border-purple-500 rounded shadow-sm px-6 py-2 relative">
     <div class="flex items-center gap-2 text-purple-500 dark:text-purple-300">
       {{ i "git-merge" "w-4 h-4" }}
       <span class="font-medium">pull request successfully merged</span
@@ -504,7 +504,7 @@
     </div>
   </div>
   {{ else if .Pull.State.IsAbandoned }}
-  <div class="bg-red-50 dark:bg-red-900 border border-red-500 rounded drop-shadow-sm px-6 py-2 relative">
+  <div class="bg-red-50 dark:bg-red-900 border border-red-500 rounded shadow-sm px-6 py-2 relative">
     <div class="flex items-center gap-2 text-red-500 dark:text-red-300">
       {{ i "git-pull-request-closed" "w-4 h-4" }}
       <span class="font-medium">This pull has been deleted (possibly by jj abandon or jj squash)</span>
@@ -515,7 +515,7 @@
 
 {{ define "resubmitStatus" }}
   {{ if .ResubmitCheck.Yes }}
-  <div class="bg-amber-50 dark:bg-amber-900 border border-amber-500 rounded drop-shadow-sm px-6 py-2 relative">
+  <div class="bg-amber-50 dark:bg-amber-900 border border-amber-500 rounded shadow-sm px-6 py-2 relative">
     <div class="flex items-center gap-2 text-amber-500 dark:text-amber-300">
       {{ i "triangle-alert" "w-4 h-4" }}
       <span class="font-medium">this branch has been updated, consider resubmitting</span>
@@ -652,7 +652,7 @@
 {{ end }}
 
 {{ define "loginPrompt" }}
-  <div class="bg-amber-50 dark:bg-amber-900 border border-amber-500 rounded drop-shadow-sm p-2 relative flex gap-2 items-center">
+  <div class="bg-amber-50 dark:bg-amber-900 border border-amber-500 rounded shadow-sm p-2 relative flex gap-2 items-center">
     <a href="/signup" class="btn-create py-0 hover:no-underline hover:text-white flex items-center gap-2">
       sign up
     </a>
diff --git a/appview/pages/templates/repo/issues/fragments/issueListing.html b/appview/pages/templates/repo/issues/fragments/issueListing.html
--- a/appview/pages/templates/repo/issues/fragments/issueListing.html
+++ b/appview/pages/templates/repo/issues/fragments/issueListing.html
@@ -1,7 +1,7 @@
 {{ define "repo/issues/fragments/issueListing" }}
   <div class="flex flex-col gap-2">
     {{ range .Issues }}
-    <div class="rounded drop-shadow-sm bg-white px-6 py-4 dark:bg-gray-800 dark:border-gray-700">
+    <div class="rounded shadow-sm bg-white px-6 py-4 dark:bg-gray-800 dark:border-gray-700">
       <div class="pb-2">
         <a
             href="/{{ $.RepoPrefix }}/issues/{{ .IssueId }}"

--
tangled.sh