From d6cc2bfcefcb202b0abade3a5f5eeb16102f1154 Mon Sep 17 00:00:00 2001
From: Emelia Smith <ThisIsMissEm@users.noreply.github.com>
Date: Fri, 17 May 2024 19:05:32 +0200
Subject: [PATCH] fix: use anchor elements for packages to allow cmd+click

---
 inertia/components/package_logo.vue           |  2 +-
 .../pages/home/components/package_card.vue    | 66 +++++++++----------
 inertia/pages/package/components/heading.vue  |  2 +-
 3 files changed, 33 insertions(+), 37 deletions(-)

diff --git a/inertia/components/package_logo.vue b/inertia/components/package_logo.vue
index 2f908ee..28f3e13 100644
--- a/inertia/components/package_logo.vue
+++ b/inertia/components/package_logo.vue
@@ -15,7 +15,7 @@ function iconPlaceholder({ category }: PackageInfo) {
 
 <template>
   <div
-    class="mt-4 items-center justify-center"
+    class="items-center justify-center"
     :class="{
       'h-12 w-12': size === '12',
       'h-16 w-16': size === '16',
diff --git a/inertia/pages/home/components/package_card.vue b/inertia/pages/home/components/package_card.vue
index ae2fee9..1cbe35f 100644
--- a/inertia/pages/home/components/package_card.vue
+++ b/inertia/pages/home/components/package_card.vue
@@ -16,57 +16,53 @@ const isV6Compatible = computed(() => props.package.compatibility?.adonis.includ
 </script>
 
 <template>
-  <Link
-    class="group card relative h-66 flex cursor-pointer gap-y-2 rounded-xl px-5 py-5"
-    :href="`/packages/${package.slug}`"
-    target="_blank"
-    as="div"
-  >
-    <div class="w-full flex items-center justify-between">
+  <div class="group card gap-y-3 relative h-66 py-5 flex rounded-xl">
+    <div class="w-full flex grow-0 items-center cursor-default px-5">
       <div class="flex gap-2">
         <Tag data-testid="package-category">{{ package.category }}</Tag>
         <Tag v-if="isV6Compatible" class="bg-primary bg-opacity70">V6 support</Tag>
       </div>
-      <i
-        class="i-icon-park-outline-share inline-block text-xs text-base10 opacity-0 transition duration-500 hover:text-white group-hover:opacity-100"
-      />
     </div>
-    <PackageLogo size="12" :package="package" />
+    <Link
+      class="flex flex-col grow gap-y-2 content-around w-full px-5"
+      :href="`/packages/${package.slug}`"
+    >
+      <PackageLogo class="mt-3" size="12" :package="package" />
 
-    <div class="flex flex-col gap-y-1">
-      <div class="pointer-events-none absolute inset-0 m-1 overflow-hidden rounded-xl">
-        <AdonisIcon
-          v-if="package.type === 'official'"
-          class="absolute top-2 scale-600 opacity-2 -right-2 -rotate-20"
-        />
-      </div>
-      <div class="flex items-baseline gap-2">
-        <p data-testid="package-name" class="line-clamp-1 text-2xl font-bold">
-          {{ package.name }}
+      <div class="flex flex-col gap-y-1">
+        <div class="pointer-events-none absolute inset-0 m-1 overflow-hidden rounded-xl">
+          <AdonisIcon
+            v-if="package.type === 'official'"
+            class="absolute top-3 scale-600 opacity-2 -right-2 -rotate-20"
+          />
+        </div>
+        <div class="flex items-baseline gap-2">
+          <p data-testid="package-name" class="line-clamp-1 text-2xl font-bold">
+            {{ package.name }}
+          </p>
+          <i
+            v-if="package.type === 'official'"
+            v-tooltip="{ content: 'Official package', distance: 10 }"
+            class="i-fluent-emoji-military-medal relative top-[2px] inline-block text-lg"
+          />
+        </div>
+        <p class="line-clamp-2 text-sm text-base10 mb-4">
+          {{ package.description }}
         </p>
-        <i
-          v-if="package.type === 'official'"
-          v-tooltip="{ content: 'Official package', distance: 10 }"
-          class="i-fluent-emoji-military-medal relative top-[2px] inline-block text-lg"
-        />
       </div>
-      <p class="line-clamp-2 text-sm text-base10">
-        {{ package.description }}
-      </p>
-    </div>
-
-    <PackageStats class="mt-4" :package="package" />
-  </Link>
+    </Link>
+    <PackageStats class="place-self-start justify-self-end px-5" :package="package" />
+  </div>
 </template>
 
 <style scoped lang="postcss">
 .card {
   @apply bg-base2;
+  /*transition: all 0.6s ease-in-out;*/
+  border: 1px solid transparent;
   flex-direction: column;
   justify-content: space-between;
   align-items: flex-start;
-  transition: all 0.6s ease-in-out;
-  border: 1px solid transparent;
 }
 </style>
 @/app/types
diff --git a/inertia/pages/package/components/heading.vue b/inertia/pages/package/components/heading.vue
index dae78e7..5751509 100644
--- a/inertia/pages/package/components/heading.vue
+++ b/inertia/pages/package/components/heading.vue
@@ -11,7 +11,7 @@ defineProps<{
 <template>
   <div class="flex flex-col text-left">
     <div class="flex flex-col items-start" md="flex-row gap-9">
-      <PackageLogo class="flex-shrink-0" size="16" :package="package" />
+      <PackageLogo class="flex-shrink-0 mt-4" size="16" :package="package" />
       <div class="flex flex-col">
         <h1 class="relative top-2 mt-1 text-5xl font-bold">{{ package.name }}</h1>
         <span class="mt-5 text-base text-base11 font-content">