From d875433d56b182c151b36ef18eeb8613d471ebc5 Mon Sep 17 00:00:00 2001
From: Spythere <spythere@protonmail.com>
Date: Fri, 8 Dec 2023 17:14:49 +0100
Subject: [PATCH] =?UTF-8?q?ulepszone=20zapami=C4=99tywanie=20zak=C5=82adek?=
 =?UTF-8?q?=20statystyk=20w=20dzienniku?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../{DailyStats.vue => JournalDailyStats.vue} |  6 +-
 .../JournalView/JournalDriverStats.vue        |  2 +
 src/components/JournalView/JournalOptions.vue |  2 -
 src/components/JournalView/JournalStats.vue   | 95 ++++++++-----------
 src/store/mainStore.ts                        |  2 -
 src/store/typings.ts                          |  2 -
 6 files changed, 44 insertions(+), 65 deletions(-)
 rename src/components/JournalView/{DailyStats.vue => JournalDailyStats.vue} (98%)

diff --git a/src/components/JournalView/DailyStats.vue b/src/components/JournalView/JournalDailyStats.vue
similarity index 98%
rename from src/components/JournalView/DailyStats.vue
rename to src/components/JournalView/JournalDailyStats.vue
index 3cfa2e2..7f6e796 100644
--- a/src/components/JournalView/DailyStats.vue
+++ b/src/components/JournalView/JournalDailyStats.vue
@@ -139,8 +139,10 @@ import { API } from '../../typings/api';
 import { Status } from '../../typings/common';
 
 export default defineComponent({
+  name: 'journal-daily-stats',
+
   mixins: [dateMixin],
-  emits: ['toggleStatsOpen'],
+  // emits: ['toggleStatsOpen'],
 
   data() {
     return {
@@ -154,7 +156,7 @@ export default defineComponent({
 
   activated() {
     this.startFetchingDailyStats();
-    this.$emit('toggleStatsOpen', true);
+    // this.$emit('toggleStatsOpen', true);
   },
 
   deactivated() {
diff --git a/src/components/JournalView/JournalDriverStats.vue b/src/components/JournalView/JournalDriverStats.vue
index ee2d2f8..1062ab6 100644
--- a/src/components/JournalView/JournalDriverStats.vue
+++ b/src/components/JournalView/JournalDriverStats.vue
@@ -59,6 +59,8 @@ import { useStore } from '../../store/mainStore';
 import { Status } from '../../typings/common';
 
 export default defineComponent({
+  name: 'journal-driver-stats',
+
   data() {
     return {
       store: useStore(),
diff --git a/src/components/JournalView/JournalOptions.vue b/src/components/JournalView/JournalOptions.vue
index 48a33ac..738e1d7 100644
--- a/src/components/JournalView/JournalOptions.vue
+++ b/src/components/JournalView/JournalOptions.vue
@@ -184,8 +184,6 @@ export default defineComponent({
   watch: {
     async 'store.driverStatsName'() {
       await this.fetchDriverStats();
-
-      // if (value) this.store.currentStatsTab = 'driver';
     },
 
     async 'searchersValues.search-driver'(value: string | undefined) {
diff --git a/src/components/JournalView/JournalStats.vue b/src/components/JournalView/JournalStats.vue
index be1d03a..23ae57d 100644
--- a/src/components/JournalView/JournalStats.vue
+++ b/src/components/JournalView/JournalStats.vue
@@ -1,27 +1,26 @@
 <template>
   <div class="journal-stats" v-if="!store.isOffline">
-    <div class="tabs">
+    <div class="stats-buttons">
       <button
-        v-for="tab in data.tabs"
-        :key="tab.name"
-        class="btn--filled"
-        :data-selected="tab.name == store.currentStatsTab && areStatsOpen"
-        :data-inactive="tab.inactive"
-        :data-disabled="tab.inactive"
-        :disabled="tab.inactive"
-        @click="onTabButtonClick(tab.name)"
+        v-for="button in data.statsButtons"
+        :key="button.name"
+        class="btn--filled btn--image"
+        :data-selected="button.name == currentStatsTab"
+        @click="onTabButtonClick(button.name)"
       >
-        {{ $t(tab.titlePath) }}
+        <img
+          v-if="button.iconName"
+          :src="`/images/icon-${button.iconName}.svg`"
+          :alt="button.iconName"
+        />
+        {{ $t(button.localeKey) }}
       </button>
     </div>
 
-    <div class="stats-tab" v-show="areStatsOpen">
+    <div class="stats-tab" v-show="currentStatsTab !== null">
       <keep-alive>
-        <JournalDailyStats
-          v-if="store.currentStatsTab == 'daily'"
-          @toggleStatsOpen="toggleStatsOpen"
-        />
-        <JournalDriverStats v-else-if="store.currentStatsTab == 'driver'" />
+        <JournalDailyStats v-if="currentStatsTab == 'journal-daily-stats'" />
+        <JournalDriverStats v-else-if="currentStatsTab == 'journal-driver-stats'" />
       </keep-alive>
     </div>
   </div>
@@ -30,67 +29,49 @@
 <script setup lang="ts">
 import { computed, onMounted, reactive, Ref, ref, watch } from 'vue';
 import { useStore } from '../../store/mainStore';
-import JournalDailyStats from './DailyStats.vue';
+import JournalDailyStats from './JournalDailyStats.vue';
 import JournalDriverStats from './JournalDriverStats.vue';
 import StorageManager from '../../managers/storageManager';
 
-// Types
-type TStatTab = 'daily' | 'driver';
+export type JournalStatsTab = 'journal-driver-stats' | 'journal-daily-stats';
 
-// Variables
 const store = useStore();
-
-const lastDailyStatsOpen = ref(false);
-const areStatsOpen = ref(false);
-const lastClickedTab: Ref<'daily' | 'driver' | null> = ref(null);
+const currentStatsTab: Ref<JournalStatsTab | null> = ref(null);
 
 let data = reactive({
-  tabs: [
+  statsButtons: [
     {
-      name: 'daily',
-      titlePath: 'journal.daily-stats-title'
+      name: 'journal-daily-stats',
+      localeKey: 'journal.daily-stats-title',
+      iconName: 'stats'
     },
     {
-      name: 'driver',
-      titlePath: 'journal.driver-stats-title'
-      // inactive: true,
+      name: 'journal-driver-stats',
+      localeKey: 'journal.driver-stats-title',
+      iconName: 'user'
     }
-  ] as { name: TStatTab; titlePath: string; inactive?: boolean }[]
+  ] as { name: JournalStatsTab; localeKey: string; iconName?: string }[]
 });
 
-// Methods
-function onTabButtonClick(tab: TStatTab) {
-  if (lastClickedTab.value == tab || !lastClickedTab.value || !areStatsOpen.value)
-    areStatsOpen.value = !areStatsOpen.value;
-
-  if (tab == 'daily') {
-    StorageManager.setBooleanValue('dailyStatsOpen', areStatsOpen.value);
-    lastDailyStatsOpen.value = areStatsOpen.value;
-  }
-
-  store.currentStatsTab = tab;
-  lastClickedTab.value = tab;
-
-  if (areStatsOpen.value == false) store.currentStatsTab = null;
-}
-
-function toggleStatsOpen(open: boolean) {
-  areStatsOpen.value = open;
+function onTabButtonClick(tab: JournalStatsTab) {
+  currentStatsTab.value = tab == currentStatsTab.value ? null : tab;
+  StorageManager.setStringValue('journalStatsTab', currentStatsTab.value ?? '');
 }
 
 watch(
   computed(() => store.driverStatsData),
-  (statsData) => {
-    store.currentStatsTab = statsData ? 'driver' : lastClickedTab.value;
-    areStatsOpen.value = statsData ? true : lastClickedTab.value !== null;
+  (newData, prevData) => {
+    currentStatsTab.value =
+      JSON.stringify(prevData) !== JSON.stringify(newData) && newData !== undefined
+        ? 'journal-driver-stats'
+        : currentStatsTab.value;
   }
 );
 
 onMounted(() => {
-  if (StorageManager.getBooleanValue('dailyStatsOpen')) {
-    areStatsOpen.value = true;
-    store.currentStatsTab = 'daily';
-  }
+  const storedTab = StorageManager.getStringValue('journalStatsTab');
+
+  if (storedTab && storedTab !== '') currentStatsTab.value = storedTab as JournalStatsTab;
 });
 </script>
 
@@ -98,7 +79,7 @@ onMounted(() => {
 @import '../../styles/JournalStats.scss';
 @import '../../styles/variables.scss';
 
-.tabs {
+.stats-buttons {
   position: relative;
 
   display: flex;
diff --git a/src/store/mainStore.ts b/src/store/mainStore.ts
index 064edf1..4a0bd1f 100644
--- a/src/store/mainStore.ts
+++ b/src/store/mainStore.ts
@@ -51,8 +51,6 @@ export const useStore = defineStore('store', {
         trains: Status.Data.Loading
       },
 
-      currentStatsTab: null,
-
       blockScroll: false,
       listenerLaunched: false,
       modalLastClickedTarget: null,
diff --git a/src/store/typings.ts b/src/store/typings.ts
index 397ac8e..1d76b2c 100644
--- a/src/store/typings.ts
+++ b/src/store/typings.ts
@@ -41,8 +41,6 @@ export interface StoreState {
 
   chosenModalTrainId?: string;
 
-  currentStatsTab: 'daily' | 'driver' | null;
-
   dataStatuses: {
     connection: Status.Data;
     sceneries: Status.Data;