From 45d19b6bfc86b00cb2e22eacc5956182dde2ad96 Mon Sep 17 00:00:00 2001
From: Rob B <computerguy440+gh@gmail.com>
Date: Sat, 6 Jan 2024 08:48:35 +0000
Subject: [PATCH 1/4] fix: add spacing at the bottom of the tags collapse
 section to avoid paginator drawing over tooltips (z index isn't helping for
 some reason)

---
 src/lib/components/mods/ModGrid.svelte     | 2 +-
 src/lib/components/utils/TagDisplay.svelte | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/lib/components/mods/ModGrid.svelte b/src/lib/components/mods/ModGrid.svelte
index 400740ea8..b075fb953 100644
--- a/src/lib/components/mods/ModGrid.svelte
+++ b/src/lib/components/mods/ModGrid.svelte
@@ -153,7 +153,7 @@
         </div>
       </div>
       {#if tagsOpen}
-        <div class="flex flex-grow flex-row flex-wrap items-center justify-center gap-1">
+        <div class="flex flex-grow flex-row flex-wrap items-center justify-center gap-1 pb-10">
           {#if $allTags.error}
             <p>Oh no... {$allTags.error.message}</p>
           {:else if !$allTags.fetching}
diff --git a/src/lib/components/utils/TagDisplay.svelte b/src/lib/components/utils/TagDisplay.svelte
index 1edf81584..0ac7d6960 100644
--- a/src/lib/components/utils/TagDisplay.svelte
+++ b/src/lib/components/utils/TagDisplay.svelte
@@ -19,6 +19,7 @@
 </div>
 
 <!-- Must use high z-index to ensure it draws on top of things like striped compatibility overlays in mod cards -->
+<!-- For some reason this still isn't able to draw over the ModGrid paginator-->
 <!-- https://floating-ui.com/docs/misc#z-index-stacking -->
 <div class="card variant-filled-surface p-4" data-popup={`popupHoverFor${tag.name}`} style="z-index: 1000">
   <p>{tag.description}.</p>

From 6417819442cccdbab251d999e73261625741adf5 Mon Sep 17 00:00:00 2001
From: Rob B <computerguy440+gh@gmail.com>
Date: Sat, 6 Jan 2024 08:49:14 +0000
Subject: [PATCH 2/4] fix: remove forced period at end of tag descriptions

---
 src/lib/components/utils/TagDisplay.svelte | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/components/utils/TagDisplay.svelte b/src/lib/components/utils/TagDisplay.svelte
index 0ac7d6960..2bfd645db 100644
--- a/src/lib/components/utils/TagDisplay.svelte
+++ b/src/lib/components/utils/TagDisplay.svelte
@@ -22,5 +22,5 @@
 <!-- For some reason this still isn't able to draw over the ModGrid paginator-->
 <!-- https://floating-ui.com/docs/misc#z-index-stacking -->
 <div class="card variant-filled-surface p-4" data-popup={`popupHoverFor${tag.name}`} style="z-index: 1000">
-  <p>{tag.description}.</p>
+  <p>{tag.description}</p>
 </div>

From d546545408a74306a21cef1a75beb43702584651 Mon Sep 17 00:00:00 2001
From: Rob B <computerguy440+gh@gmail.com>
Date: Sat, 6 Jan 2024 08:50:21 +0000
Subject: [PATCH 3/4] chore: switch tag data binding to use tag id

---
 src/lib/components/utils/TagDisplay.svelte | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/lib/components/utils/TagDisplay.svelte b/src/lib/components/utils/TagDisplay.svelte
index 2bfd645db..7419d1f4b 100644
--- a/src/lib/components/utils/TagDisplay.svelte
+++ b/src/lib/components/utils/TagDisplay.svelte
@@ -6,7 +6,7 @@
 
   $: popupHover = {
     event: popupTriggerEvent,
-    target: `popupHoverFor${tag.name}`,
+    target: `popupHoverFor${tag.id}`,
     placement: 'bottom-start'
   } satisfies PopupSettings;
 </script>
@@ -21,6 +21,6 @@
 <!-- Must use high z-index to ensure it draws on top of things like striped compatibility overlays in mod cards -->
 <!-- For some reason this still isn't able to draw over the ModGrid paginator-->
 <!-- https://floating-ui.com/docs/misc#z-index-stacking -->
-<div class="card variant-filled-surface p-4" data-popup={`popupHoverFor${tag.name}`} style="z-index: 1000">
+<div class="card variant-filled-surface p-4" data-popup={`popupHoverFor${tag.id}`} style="z-index: 1000">
   <p>{tag.description}</p>
 </div>

From c26f6393f4086ca835a0608d06c8dd8981c824e0 Mon Sep 17 00:00:00 2001
From: Rob B <computerguy440+gh@gmail.com>
Date: Sat, 6 Jan 2024 09:02:19 +0000
Subject: [PATCH 4/4] chore: fix hover-tags saying you should click on them in
 browser title-tooltip

---
 src/lib/components/utils/TagDisplay.svelte | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/components/utils/TagDisplay.svelte b/src/lib/components/utils/TagDisplay.svelte
index 7419d1f4b..373d0e7a5 100644
--- a/src/lib/components/utils/TagDisplay.svelte
+++ b/src/lib/components/utils/TagDisplay.svelte
@@ -14,7 +14,7 @@
 <div
   class="lowercase text-neutral-300 [&>*]:pointer-events-none"
   use:popup={popupHover}
-  title="Click for tag information">
+  title={popupTriggerEvent === 'click' ? 'Click for tag information' : ''}>
   <span class="text-orange-500">#</span>{tag.name}
 </div>