From cba119b3fc4e1bf312cd93a0f7a46f763a5cd7b9 Mon Sep 17 00:00:00 2001
From: Julien Calixte <julienc@bam.tech>
Date: Sat, 15 Apr 2023 18:37:18 +0200
Subject: [PATCH] better styling for task view with tags

---
 .../record/components/TaskRecordList.vue      | 40 +++++++++++--------
 src/modules/task/components/TaskList.vue      |  2 +-
 src/views/task/TaskView.vue                   | 30 ++++++++++----
 3 files changed, 48 insertions(+), 24 deletions(-)

diff --git a/src/modules/record/components/TaskRecordList.vue b/src/modules/record/components/TaskRecordList.vue
index 2ffa21d..0c35adc 100644
--- a/src/modules/record/components/TaskRecordList.vue
+++ b/src/modules/record/components/TaskRecordList.vue
@@ -1,5 +1,6 @@
 <script setup lang="ts">
 import { createUuid } from '@/shared/create-uuid'
+import { computed } from 'vue'
 import { useTaskRecordStore } from '../stores/useTaskRecordStore'
 import TaskRecordLink from './TaskRecordLink.vue'
 
@@ -9,25 +10,32 @@ const props = defineProps<{
 
 const recordStore = useTaskRecordStore()
 
-const records = recordStore.getTaskRecords(props.taskId)
+const recordsFromLastToFirst = computed(() =>
+  recordStore
+    .getTaskRecords(props.taskId)
+    .sort((a, b) => (a.start > b.start ? -1 : 1))
+)
 const newRecordId = createUuid()
 </script>
 
 <template>
-  <div class="content">
-    <ol v-if="records.length" class="task-record-list">
-      <li v-for="record in records" :key="record.id">
-        <task-record-link :record="record" />
-      </li>
-    </ol>
-    <div v-else>No record yet</div>
+  <div>
+    <div class="content">
+      <h3 class="subtitle is-4">Records</h3>
+      <ol v-if="recordsFromLastToFirst.length" class="task-record-list">
+        <li v-for="record in recordsFromLastToFirst" :key="record.id">
+          <task-record-link :record="record" />
+        </li>
+      </ol>
+      <div v-else>No record yet</div>
+    </div>
+    <router-link
+      :to="{
+        name: 'record-view',
+        params: { taskId, recordId: newRecordId }
+      }"
+      class="button is-primary is-light"
+      >start a new record</router-link
+    >
   </div>
-  <router-link
-    :to="{
-      name: 'record-view',
-      params: { taskId, recordId: newRecordId }
-    }"
-    class="button is-primary is-light"
-    >start a new record</router-link
-  >
 </template>
diff --git a/src/modules/task/components/TaskList.vue b/src/modules/task/components/TaskList.vue
index d92dea6..41680cd 100644
--- a/src/modules/task/components/TaskList.vue
+++ b/src/modules/task/components/TaskList.vue
@@ -14,7 +14,7 @@ const taskStore = useTaskStore()
           class="button is-link is-outlined"
           >{{ task.title }}</router-link
         >
-        <span> {{ task.totalEstimation }} minutes </span>
+        <span class="tag">{{ task.totalEstimation }} minutes</span>
         <span>{{ formatDate(task.date) }}</span>
       </li>
     </ul>
diff --git a/src/views/task/TaskView.vue b/src/views/task/TaskView.vue
index ab21b99..be21e36 100644
--- a/src/views/task/TaskView.vue
+++ b/src/views/task/TaskView.vue
@@ -24,14 +24,30 @@ const task = computed(() => taskStore.getTask(props.id))
       class="button is-link"
       >user story link</a
     >
-    <div class="content is-large">
-      <ol>
-        <li v-for="step in task.steps" :key="step.id">
-          <div>{{ step.title }} | {{ step.estimation }}</div>
-        </li>
-      </ol>
+    <div class="columns">
+      <div class="column content is-large">
+        <h3 class="subtitle is-4">Tasks</h3>
+        <ol>
+          <li v-for="step in task.steps" :key="step.id">
+            <div class="step-item">
+              <span>{{ step.title }}</span>
+              <span class="tag">{{ step.estimation }} minutes</span>
+            </div>
+          </li>
+        </ol>
+      </div>
+      <task-record-list class="column" :task-id="id" />
     </div>
-    <task-record-list :task-id="id" />
   </div>
   <div v-else>Task not found</div>
 </template>
+
+<style scoped>
+.step-item {
+  display: flex;
+  align-items: center;
+  gap: 1rem;
+  justify-content: space-between;
+  max-width: 400px;
+}
+</style>