From 173544417630e8360ee4c52979538852a1e28469 Mon Sep 17 00:00:00 2001
From: Spythere <spythere@protonmail.com>
Date: Fri, 20 Feb 2026 01:58:29 +0100
Subject: [PATCH] refactor(profile): data loading indicators

---
 .../PlayerProfileView/ProfileHistoryList.vue  | 25 +++++--
 src/views/PlayerProfileView.vue               | 67 ++++++-------------
 2 files changed, 40 insertions(+), 52 deletions(-)

diff --git a/src/components/PlayerProfileView/ProfileHistoryList.vue b/src/components/PlayerProfileView/ProfileHistoryList.vue
index b699ffc..7f38a8a 100644
--- a/src/components/PlayerProfileView/ProfileHistoryList.vue
+++ b/src/components/PlayerProfileView/ProfileHistoryList.vue
@@ -14,11 +14,14 @@
     </div>
 
     <div class="history-list-box">
-      <div v-if="combinedJournal.length == 0" class="no-recent-history text--grayed">
+      <Loading v-if="journalDataStatus == Status.Data.Loading" />
+
+      <div v-else-if="combinedJournal.length == 0" class="no-recent-history">
         {{ t('profile.list.no-recent-history') }}
       </div>
 
       <router-link
+        v-else
         v-for="entry in combinedJournal"
         :to="
           'trainNo' in entry.value
@@ -110,6 +113,8 @@ import { API } from '../../typings/api';
 import { useI18n } from 'vue-i18n';
 import { useApiStore } from '../../store/apiStore';
 import { useRoute } from 'vue-router';
+import { Status } from '../../typings/common';
+import Loading from '../Global/Loading.vue';
 
 type JournalEntryType = 'Timetable' | 'Dispatcher' | 'IssuedTimetable';
 
@@ -129,7 +134,9 @@ const { t } = useI18n();
 const apiStore = useApiStore();
 const route = useRoute();
 
+const playerId = ref(-1);
 const playerJournal = ref<API.PlayerJournal.Data | null>(null);
+const journalDataStatus = ref(Status.Data.Initialized);
 
 const activeFilterTypes = reactive<Record<JournalEntryType, boolean>>({
   Timetable: true,
@@ -196,21 +203,29 @@ function toggleFilter(filterType: JournalEntryType) {
 }
 
 async function fetchPlayerJournal() {
-  const playerId = route.query.playerId?.toString();
+  const queryPlayerId = Number(route.query.playerId) || -1;
 
-  if (!apiStore.client || !playerId) return null;
+  if (!apiStore.client || !queryPlayerId) return;
+
+  if (queryPlayerId != playerId.value) {
+    journalDataStatus.value = Status.Data.Loading;
+  }
+
+  playerId.value = queryPlayerId;
 
   try {
     const response = await apiStore.client.get<API.PlayerJournal.Data>('api/getPlayerJournal', {
       params: {
-        playerId: playerId,
+        playerId: queryPlayerId,
         dateScope: '30d'
       }
     });
 
     playerJournal.value = response.data;
+    journalDataStatus.value = Status.Data.Loaded;
   } catch (error) {
     console.error(error);
+    journalDataStatus.value = Status.Data.Error;
   }
 
   return null;
@@ -255,6 +270,7 @@ async function fetchPlayerJournal() {
 
 .history-list-box {
   padding: 0 0.5em;
+  position: relative;
 }
 
 .history-list-box > a {
@@ -277,6 +293,7 @@ async function fetchPlayerJournal() {
   padding: 1em;
   font-size: 1.25em;
   font-weight: bold;
+  color: #aaa;
 }
 
 .entry-top-date {
diff --git a/src/views/PlayerProfileView.vue b/src/views/PlayerProfileView.vue
index bdd351e..16156c0 100644
--- a/src/views/PlayerProfileView.vue
+++ b/src/views/PlayerProfileView.vue
@@ -9,9 +9,7 @@
       </div>
     </div>
 
-    <div v-else-if="playerDataStatus == Status.Data.Loading">
-      <Loading />
-    </div>
+    <Loading v-else-if="playerDataStatus == Status.Data.Loading" />
 
     <div class="no-data-found" v-else>
       <div>
@@ -25,8 +23,8 @@
 <script lang="ts" setup>
 import axios from 'axios';
 
-import { computed, onMounted, ref, watch } from 'vue';
-import { useRoute } from 'vue-router';
+import { computed, onActivated, onMounted, ref, watch } from 'vue';
+import { onBeforeRouteUpdate, useRoute } from 'vue-router';
 import { useApiStore } from '../store/apiStore';
 import { API, Td2API } from '../typings/api';
 import { useI18n } from 'vue-i18n';
@@ -42,70 +40,43 @@ const { t } = useI18n();
 const apiStore = useApiStore();
 const route = useRoute();
 
+const playerId = ref(-1);
 const playerName = ref('');
 
 const playerInfo = ref<API.PlayerInfo.Data | null>(null);
 const playerDataStatus = ref(Status.Data.Initialized);
 
-watch(
-  computed(() => route.query.playerId),
-  () => {
-    fetchAllData();
-  }
-);
-
-onMounted(() => {
-  fetchAllData();
+onActivated(() => {
+  fetchPlayerData();
 });
 
-async function fetchAllData() {
-  const playerId = route.query.playerId?.toString();
+async function fetchPlayerData() {
+  const queryPlayerId = Number(route.query.playerId) || -1;
 
-  playerInfo.value = null;
-  playerDataStatus.value = Status.Data.Loading;
+  if (!apiStore.client || !queryPlayerId) return;
 
-  if (!playerId) {
-    playerDataStatus.value = Status.Data.Loaded;
-    return;
+  if (queryPlayerId != playerId.value) {
+    playerDataStatus.value = Status.Data.Loading;
   }
 
-  const playerInfoResponse = await fetchPlayerInfoData(playerId);
-
-  if (!playerInfoResponse) {
-    playerDataStatus.value = Status.Data.Loaded;
-    return;
-  }
-
-  playerName.value =
-    playerInfoResponse.driverStats.driverName ||
-    playerInfoResponse.dispatcherStats.dispatcherName ||
-    '';
-
-  if (!playerName.value) {
-    playerDataStatus.value = Status.Data.Loaded;
-    return;
-  }
-
-  playerInfo.value = playerInfoResponse;
-  playerDataStatus.value = Status.Data.Loaded;
-}
-
-async function fetchPlayerInfoData(playerId: string) {
-  if (!apiStore.client || !playerId) return null;
+  playerId.value = queryPlayerId;
 
   try {
     const response = await apiStore.client.get<API.PlayerInfo.Data>('api/getPlayerInfo', {
       params: {
-        playerId: playerId
+        playerId: queryPlayerId
       }
     });
 
-    return response.data;
+    playerName.value =
+      response.data.driverStats.driverName || response.data.dispatcherStats.dispatcherName || '';
+
+    playerInfo.value = response.data || null;
+    playerDataStatus.value = Status.Data.Loaded;
   } catch (error) {
     console.error(error);
+    playerDataStatus.value = Status.Data.Error;
   }
-
-  return null;
 }
 </script>