From fb48f64ed88fc3d4751cdf71644d889bfc0f7ee1 Mon Sep 17 00:00:00 2001
From: Klemek <klemek.dev@proton.me>
Date: Thu, 16 Jul 2026 11:28:18 +0200
Subject: [PATCH] fix: paddings and margins

---
 resources/ts/App.vue                   | 11 ++-
 resources/ts/components/RecentItem.vue |  7 +-
 resources/ts/views/HomeView.vue        | 29 +++++---
 resources/ts/views/ListView.vue        | 97 +++++++++++++-------------
 resources/ts/views/NotFoundView.vue    | 14 ++--
 5 files changed, 87 insertions(+), 71 deletions(-)

diff --git a/resources/ts/App.vue b/resources/ts/App.vue
index a6e6c4c..559a184 100644
--- a/resources/ts/App.vue
+++ b/resources/ts/App.vue
@@ -25,12 +25,17 @@ onMounted(() => {
         <div class="hero bg-hero grow">
             <div class="hero-content text-center p-0">
                 <div
-                    class="max-w-md min-w-96 bg-base-100 rounded-box shadow-md"
+                    class="max-w-md min-w-96 bg-base-100 rounded-box shadow-md p-4"
                 >
                     <h1
-                        class="p-4 pb-2 text-4xl font-bold opacity-60 tracking-wide select-none"
+                        class="text-4xl font-bold opacity-60 tracking-wide select-none"
                     >
-                        <list-todo-icon :stroke-width="3" :size="32" class="inline" /> Tout Doux
+                        <list-todo-icon
+                            :stroke-width="3"
+                            :size="32"
+                            class="inline"
+                        />
+                        Tout Doux
                     </h1>
                     <router-view />
                 </div>
diff --git a/resources/ts/components/RecentItem.vue b/resources/ts/components/RecentItem.vue
index c4ab976..81b96a3 100644
--- a/resources/ts/components/RecentItem.vue
+++ b/resources/ts/components/RecentItem.vue
@@ -1,15 +1,14 @@
 <script setup lang="ts">
 import { ArrowBigRightDashIcon } from "@lucide/vue";
 
-defineProps<{list: string}>();
+defineProps<{ list: string }>();
 </script>
 
 <template>
     <router-link
         :to="`/${list}`"
-        class="w-full flex gap-2 mb-2"
+        class="w-full flex gap-2 mb-2 hover:underline"
     >
-        <arrow-big-right-dash-icon :size="16" class="inline" />
-        <span class="underline">{{ list }}</span>
+        <arrow-big-right-dash-icon :size="16" class="inline" /> {{ list }}
     </router-link>
 </template>
diff --git a/resources/ts/views/HomeView.vue b/resources/ts/views/HomeView.vue
index 118c8aa..26a07c2 100644
--- a/resources/ts/views/HomeView.vue
+++ b/resources/ts/views/HomeView.vue
@@ -24,18 +24,18 @@ function onInput() {
 }
 
 function updateTitle() {
-    document.title = 'Tout Doux';
+    document.title = "Tout Doux";
 }
 
-onBeforeMount(updateTitle)
+onBeforeMount(updateTitle);
 onBeforeRouteUpdate(updateTitle);
 </script>
 
 <template>
-    <div
-        class="p-4 italic text-xl font-extralight"
-    >{{ $t('home') }}<br/>{{ $t('call_to_action') }}</div>
-    <div class="flex gap-2 p-4">
+    <div class="italic mt-4 text-xl font-extralight">
+        {{ $t("home") }}<br />{{ $t("call_to_action") }}
+    </div>
+    <div class="flex gap-2 mt-4">
         <input
             v-model="listNameInput"
             type="text"
@@ -47,7 +47,7 @@ onBeforeRouteUpdate(updateTitle);
             <dices-icon />
         </button>
     </div>
-    <div class="w-full">
+    <div class="mt-4">
         <router-link
             class="btn btn-lg"
             :class="listNameInput.trim().length === 0 ? 'btn-disabled' : ''"
@@ -57,14 +57,21 @@ onBeforeRouteUpdate(updateTitle);
             {{ $t("home_button") }}
         </router-link>
     </div>
-    <div v-if="recentLists.length" class="collapse collapse-arrow font-light text-sm">
-        <input id="recent-dropdown" type="checkbox" />
-        <div class="collapse-title">
+    <div
+        v-if="recentLists.length"
+        class="mt-4 -mb-4 lh collapse collapse-arrow font-light text-sm"
+    >
+        <input id="recent-dropdown p-0" type="checkbox" />
+        <div class="collapse-title p-0 content-center">
             <history-icon class="inline" :size="16" />
             {{ $t("recent") }}
         </div>
         <div class="collapse-content">
-            <recent-item v-for="(list, i) in recentLists" :key="`recent-${i}`" :list="list" />
+            <recent-item
+                v-for="(list, i) in recentLists"
+                :key="`recent-${i}`"
+                :list="list"
+            />
         </div>
     </div>
 </template>
diff --git a/resources/ts/views/ListView.vue b/resources/ts/views/ListView.vue
index 00330f0..b6f5f34 100644
--- a/resources/ts/views/ListView.vue
+++ b/resources/ts/views/ListView.vue
@@ -134,19 +134,21 @@ function onChangeSortReverse() {
 }
 
 function onShare() {
-    shareLink(
-        `Tout Doux - ${list.value ?? ""}`,
-        i18n.t("share_text"),
-        `${document.location.href.split("?")[0] ?? ""}?lang=${i18n.locale.value}`,
-    )
-        .then((viaShare) => {
-            if (!viaShare) {
-                alertSuccess(i18n.t("alerts.share_clipboard"));
-            }
-        })
-        .catch(() => {
-            alertError(i18n.t("alerts.share_error"));
-        });
+    if (!empty.value) {
+        shareLink(
+            `Tout Doux - ${list.value ?? ""}`,
+            i18n.t("share_text"),
+            `${document.location.href.split("?")[0] ?? ""}?lang=${i18n.locale.value}`,
+        )
+            .then((viaShare) => {
+                if (!viaShare) {
+                    alertSuccess(i18n.t("alerts.share_clipboard"));
+                }
+            })
+            .catch(() => {
+                alertError(i18n.t("alerts.share_error"));
+            });
+    }
 }
 
 function updateTitle() {
@@ -167,11 +169,32 @@ onBeforeRouteUpdate((to) => {
 </script>
 
 <template>
-    <div class="list-row italic text-sm font-extralight self-center">
+    <div
+        class="list-row italic text-sm font-extralight self-center"
+        :class="!empty ? 'cursor-pointer hover:underline' : ''"
+        :title="!empty ? $t('share_button') : ''"
+        @click="onShare"
+    >
         {{ list }}
-        <span v-if="taskCount">({{ completedCount }} / {{ taskCount }})</span>
+        <span v-if="!empty"
+            >({{ completedCount }} / {{ taskCount }})&nbsp;<share-2-icon
+                class="inline"
+                :size="16"
+                :stroke-width="1"
+        /></span>
     </div>
-    <ul class="list">
+    <div v-if="!loading && empty" class="mt-4 mb-4 italic text-xl font-extralight">
+        <search-icon class="inline" />
+        <span v-html="$t('empty_hint')"></span>
+    </div>
+    <div
+        v-else-if="!loading && allCompleted && separateCompleted"
+        class="mt-4 mb-4 italic text-xl font-extralight"
+    >
+        <check-check-icon class="inline" />
+        <span v-html="$t('all_completed_hint')"></span>
+    </div>
+    <ul v-else class="list">
         <template v-for="(task, i) in tasks" :key="task.id">
             <li v-if="tasks[i] && (!separateCompleted || !taskCompleted(task))">
                 <task-item
@@ -184,35 +207,17 @@ onBeforeRouteUpdate((to) => {
                 />
             </li>
         </template>
-        <li
-            v-if="!loading && empty"
-            class="list-row italic text-xl font-extralight self-center"
-        >
-            <span
-                ><search-icon class="inline" />
-                <span v-html="$t('empty_hint')"></span
-            ></span>
-        </li>
-        <li
-            v-else-if="!loading && allCompleted && separateCompleted"
-            class="list-row italic text-xl font-extralight self-center"
-        >
-            <span
-                ><check-check-icon class="inline" />
-                <span v-html="$t('all_completed_hint')"></span
-            ></span>
-        </li>
     </ul>
     <div
         v-if="separateCompleted && hasCompleted"
-        class="collapse collapse-arrow"
+        class="collapse collapse-arrow mb-4"
     >
-        <input id="completed-dropdown" type="checkbox" />
-        <div class="collapse-title mb-0">
+        <input id="completed-dropdown p-0" type="checkbox" />
+        <div class="collapse-title p-0 content-center">
             <circle-check-big-icon class="inline" :size="24" />
             {{ $t("completed") }} ({{ completedCount }})
         </div>
-        <div class="collapse-content p-0">
+        <div class="collapse-content p-0 -mb-4">
             <ul class="list">
                 <template
                     v-for="(task, i) in tasks"
@@ -231,7 +236,7 @@ onBeforeRouteUpdate((to) => {
             </ul>
         </div>
     </div>
-    <div class="flex gap-2 p-4">
+    <div class="flex gap-2">
         <input
             v-model="taskInput"
             type="text"
@@ -248,14 +253,12 @@ onBeforeRouteUpdate((to) => {
             <circle-plus-icon />
         </button>
     </div>
-    <div v-if="!empty">
-        <button class="btn btn-sm px-4" @click="onShare">
-            <share-2-icon class="inline" :size="16" /> {{ $t("share_button") }}
-        </button>
-    </div>
-    <div v-if="!empty" class="collapse collapse-arrow font-light text-sm">
-        <input id="options-dropdown" type="checkbox" />
-        <div class="collapse-title">
+    <div
+        v-if="!empty"
+        class="mt-4 -mb-4 collapse collapse-arrow font-light text-sm"
+    >
+        <input id="options-dropdown p-0" type="checkbox" />
+        <div class="collapse-title p-0 content-center">
             <settings-2-icon class="inline" :size="16" />
             {{ $t("options.title") }}
         </div>
diff --git a/resources/ts/views/NotFoundView.vue b/resources/ts/views/NotFoundView.vue
index 27a1ebd..ba662c6 100644
--- a/resources/ts/views/NotFoundView.vue
+++ b/resources/ts/views/NotFoundView.vue
@@ -4,18 +4,20 @@ import { onBeforeMount } from "vue";
 import { onBeforeRouteUpdate } from "vue-router";
 
 function updateTitle() {
-    document.title = 'Tout Doux - Not Found';
+    document.title = "Tout Doux - Not Found";
 }
 
-onBeforeMount(updateTitle)
+onBeforeMount(updateTitle);
 onBeforeRouteUpdate(updateTitle);
 </script>
 
 <template>
-    <div class="italic text-xl font-extralight mb-4">
-        <circle-question-mark-icon class="inline" /> {{ $t('not_found') }}
+    <div class="italic text-xl font-extralight mt-4">
+        <circle-question-mark-icon class="inline" /> {{ $t("not_found") }}
     </div>
-    <div class="w-full">
-        <router-link to="/" class="btn mb-4 px-4">{{ $t('not_found_button') }}</router-link>
+    <div class="w-full mt-4">
+        <router-link to="/" class="btn">{{
+            $t("not_found_button")
+        }}</router-link>
     </div>
 </template>