From 9df9b9a2df78cc916266fe8b78fdb0461417f914 Mon Sep 17 00:00:00 2001
From: Benjamin Canac <canacb1@gmail.com>
Date: Tue, 21 Nov 2023 23:02:35 +0100
Subject: [PATCH] docs(releases): improve page

---
 docs/components/releases/ReleasesItem.vue | 12 ++++----
 docs/pages/releases.vue                   | 34 ++++++++++-------------
 2 files changed, 21 insertions(+), 25 deletions(-)

diff --git a/docs/components/releases/ReleasesItem.vue b/docs/components/releases/ReleasesItem.vue
index 5d79d03ce2..01b671d1cb 100644
--- a/docs/components/releases/ReleasesItem.vue
+++ b/docs/components/releases/ReleasesItem.vue
@@ -6,19 +6,19 @@
     <NuxtLink v-if="date.release" :to="`https://github.com/nuxt/ui/releases/tag/${date.release.name}`" target="_blank" class="text-gray-900 dark:text-white font-bold text-3xl mt-2 group hover:text-primary-500 dark:hover:text-primary-400 transition-[color]">
       {{ date.release.name }}
     </NuxtLink>
-    <div v-else-if="date.pulls?.length" class="mt-2 space-y-1">
-      <div v-for="(pull, index) in date.pulls" :key="index" class="text-sm/6 break-all lg:break-words">
+    <ul v-else-if="date.pulls?.length" class="mt-2 space-y-1 text-gray-600 dark:text-gray-300">
+      <li v-for="pull in date.pulls" :key="pull.id" class="text-sm/6 break-all">
         <NuxtLink :to="`https://github.com/${pull.user.login}`" target="_blank" class="text-gray-900 dark:text-white transition-colors inline-flex items-center gap-1 rounded-full bg-gray-100/50 dark:bg-gray-800/50 dark:hover:bg-gray-800 p-0.5 pr-1 ring-1 ring-gray-300 dark:ring-gray-700 text-xs font-medium flex-shrink-0 align-middle">
           <UAvatar :src="`https://github.com/${pull.user.login}.png`" size="3xs" />
 
           {{ pull.user.login }}
         </NuxtLink>
 
-        pushed <NuxtLink :to="pull.html_url" target="_blank" class="font-medium text-gray-900 dark:text-white hover:text-primary-500 dark:hover:text-primary-400 transition-[color]">
+        pushed <NuxtLink :to="pull.html_url" target="_blank" class="font-medium text-gray-700 dark:text-gray-200 hover:text-primary-500 dark:hover:text-primary-400 transition-[color]">
           #{{ pull.number }} {{ pull.title }}
         </NuxtLink>
-      </div>
-    </div>
+      </li>
+    </ul>
   </div>
 </template>
 
@@ -32,6 +32,7 @@ defineProps<{
       name: string
     }
     pulls?: {
+      id: number
       number: number
       title: string
       html_url: string
@@ -48,7 +49,6 @@ const targetIsVisible = ref(false)
 useIntersectionObserver(target, ([{ isIntersecting }]) => {
   targetIsVisible.value = isIntersecting
 }, {
-  threshold: 1,
   rootMargin: '-68px 0px -68px 0px'
 })
 </script>
diff --git a/docs/pages/releases.vue b/docs/pages/releases.vue
index 6919da51bb..2f86af74a6 100644
--- a/docs/pages/releases.vue
+++ b/docs/pages/releases.vue
@@ -1,35 +1,31 @@
 <template>
-  <UMain>
+  <div class="relative px-4 sm:px-6 lg:px-8">
     <div class="stars w-screen fixed pointer-events-none inset-x-0 -top-[--header-height] bottom-0 opacity-75">
       <div class="h-px w-px rounded-full bg-transparent" />
       <div class="h-px w-px rounded-full bg-transparent" />
       <div class="h-px w-px rounded-full bg-transparent" />
     </div>
 
-    <UContainer>
-      <UPageHero :title="page.title" :description="page.description" :links="page.links" align="center" />
+    <UPageHero :title="page.title" :description="page.description" :links="page.links" align="center" />
 
-      <UPage>
-        <UPageBody>
-          <div class="h-[96px] w-0.5 bg-gray-200 dark:bg-gray-800 mx-auto rounded-t-full" />
+    <UPageBody>
+      <div class="h-[96px] w-0.5 bg-gray-200 dark:bg-gray-800 mx-auto rounded-t-full" />
 
-          <div v-for="(date, index) in dates" :key="index" class="relative py-3 min-h-[24px] flex items-center justify-center">
-            <div class="h-full w-0.5 bg-gray-200 dark:bg-gray-800 absolute top-0 inset-x-[50%] -ml-[1px] flex-shrink-0" />
+      <div v-for="(date, index) in dates" :key="index" class="relative py-3 min-h-[24px] flex items-center justify-center">
+        <div class="h-full w-0.5 bg-gray-200 dark:bg-gray-800 absolute top-0 inset-x-[50%] -ml-[1px] flex-shrink-0" />
 
-            <template v-if="date.release || date.pulls?.length || isToday(date.day)">
-              <div class="flex items-start gap-8 relative w-[50%]" :class="index % 2 === 0 ? 'translate-x-[50%] -ml-2' : '-translate-x-[50%] ml-2 flex-row-reverse'">
-                <div class="h-[8px] w-[8px] bg-gray-400 dark:bg-gray-400 rounded-full z-[1] mt-2 ring-2 ring-gray-300 dark:ring-gray-600 flex-shrink-0" />
+        <template v-if="date.release || date.pulls?.length || isToday(date.day)">
+          <div class="flex items-start gap-8 relative w-[50%]" :class="index % 2 === 0 ? 'translate-x-[50%] -ml-2' : '-translate-x-[50%] ml-2 flex-row-reverse'">
+            <div class="h-[8px] w-[8px] bg-gray-400 dark:bg-gray-400 rounded-full z-[1] mt-2 ring-2 ring-gray-300 dark:ring-gray-600 flex-shrink-0" />
 
-                <ReleasesItem :date="date" :class="index % 2 === 0 ? '' : 'text-right'" />
-              </div>
-            </template>
+            <ReleasesItem :date="date" :class="index % 2 === 0 ? '' : 'text-right'" />
           </div>
+        </template>
+      </div>
 
-          <div class="h-[96px] w-0.5 bg-gray-200 dark:bg-gray-800 mx-auto rounded-b-full" />
-        </UPageBody>
-      </UPage>
-    </UContainer>
-  </UMain>
+      <div class="h-[96px] w-0.5 bg-gray-200 dark:bg-gray-800 mx-auto rounded-b-full" />
+    </UPageBody>
+  </div>
 </template>
 
 <script setup lang="ts">