Client - use nested routes for user profile

2021-10-23 20:04:38 +02:00 · 2021-10-23 20:04:38 +02:00 · 3bcc93bb1a
commit 3bcc93bb1a
parent d072189936
8 changed files with 215 additions and 173 deletions
--- a/fittrackee_client/src/components/User/ProfileDisplay/UserHeader.vue
+++ b/fittrackee_client/src/components/User/ProfileDisplay/UserHeader.vue
@ -0,0 +1,121 @@
+<template>
+  <div class="box user-header">
+    <UserPicture :user="user" />
+    <div class="user-details">
+      <div class="user-name">{{ user.username }}</div>
+      <div class="user-stats">
+        <div class="user-stat">
+          <span class="stat-number">{{ user.nb_workouts }}</span>
+          <span class="stat-label">
+            {{ $t('workouts.WORKOUT', user.nb_workouts) }}
+          </span>
+        </div>
+        <div class="user-stat">
+          <span class="stat-number">{{
+            Number(user.total_distance).toFixed(0)
+          }}</span>
+          <span class="stat-label">km</span>
+        </div>
+        <div class="user-stat hide-small">
+          <span class="stat-number">{{ user.nb_sports }}</span>
+          <span class="stat-label">
+            {{ $t('workouts.SPORT', user.nb_sports) }}
+          </span>
+        </div>
+      </div>
+    </div>
+  </div>
+</template>
+
+<script lang="ts">
+  import { PropType, computed, defineComponent } from 'vue'
+
+  import UserPicture from '@/components/User/UserPicture.vue'
+  import { IAuthUserProfile } from '@/types/user'
+  import { getApiUrl } from '@/utils'
+
+  export default defineComponent({
+    name: 'ProfileDisplay',
+    components: {
+      UserPicture,
+    },
+    props: {
+      user: {
+        type: Object as PropType<IAuthUserProfile>,
+        required: true,
+      },
+    },
+    setup(props) {
+      return {
+        authUserPictureUrl: computed(() =>
+          props.user.picture
+            ? `${getApiUrl()}/users/${
+                props.user.username
+              }/picture?${Date.now()}`
+            : ''
+        ),
+      }
+    },
+  })
+</script>
+
+<style lang="scss" scoped>
+  @import '~@/scss/base.scss';
+
+  .user-header {
+    display: flex;
+    align-items: stretch;
+
+    .user-details {
+      flex-grow: 1;
+      padding: $default-padding;
+      display: flex;
+      flex-direction: column;
+      align-items: center;
+
+      .user-name {
+        font-size: 2em;
+        height: 60%;
+      }
+
+      .user-stats {
+        display: flex;
+        gap: $default-padding * 4;
+        .user-stat {
+          display: flex;
+          flex-direction: column;
+          align-items: center;
+          padding-top: $default-padding;
+          .stat-number,
+          .stat-label {
+            padding: 0 $default-padding * 0.5;
+          }
+          .stat-number {
+            font-weight: bold;
+            font-size: 1.5em;
+          }
+        }
+      }
+
+      @media screen and (max-width: $x-small-limit) {
+        .user-name {
+          font-size: 1.5em;
+        }
+
+        .user-stats {
+          gap: $default-padding * 2;
+          .user-stat {
+            .stat-number {
+              font-weight: bold;
+              font-size: 1.2em;
+            }
+
+            &.hide-small {
+              display: none;
+            }
+          }
+        }
+      }
+    }
+  }
+</style>
--- a/fittrackee_client/src/components/User/ProfileDisplay/index.vue
+++ b/fittrackee_client/src/components/User/ProfileDisplay/index.vue
@ -1,55 +1,24 @@
 <template>
  <div id="user-profile">
-    <div class="box user-header">
-      <UserPicture :user="user" />
-      <div class="user-details">
-        <div class="user-name">{{ user.username }}</div>
-        <div class="user-stats">
-          <div class="user-stat">
-            <span class="stat-number">{{ user.nb_workouts }}</span>
-            <span class="stat-label">
-              {{ $t('workouts.WORKOUT', user.nb_workouts) }}
-            </span>
-          </div>
-          <div class="user-stat">
-            <span class="stat-number">{{
-              Number(user.total_distance).toFixed(0)
-            }}</span>
-            <span class="stat-label">km</span>
-          </div>
-          <div class="user-stat hide-small">
-            <span class="stat-number">{{ user.nb_sports }}</span>
-            <span class="stat-label">
-              {{ $t('workouts.SPORT', user.nb_sports) }}
-            </span>
-          </div>
-        </div>
-      </div>
-    </div>
+    <UserHeader :user="user" />
    <div class="box">
      <UserProfileTabs :tabs="tabs" :selectedTab="tab" :edition="false" />
-      <UserInfos :user="user" v-if="tab === 'PROFILE'" />
-      <UserPreferences :user="user" v-if="tab === 'PREFERENCES'" />
+      <router-view :user="user"></router-view>
    </div>
  </div>
 </template>

 <script lang="ts">
-  import { ComputedRef, PropType, computed, defineComponent } from 'vue'
+  import { PropType, defineComponent } from 'vue'

-  import UserInfos from '@/components/User/ProfileDisplay/UserInfos.vue'
-  import UserPreferences from '@/components/User/ProfileDisplay/UserPreferences.vue'
-  import UserPicture from '@/components/User/UserPicture.vue'
+  import UserHeader from '@/components/User/ProfileDisplay/UserHeader.vue'
  import UserProfileTabs from '@/components/User/UserProfileTabs.vue'
  import { IAuthUserProfile } from '@/types/user'
-  import { getApiUrl } from '@/utils'

  export default defineComponent({
    name: 'ProfileDisplay',
    components: {
-      UserInfos,
-      UserPicture,
-      UserPreferences,
+      UserHeader,
      UserProfileTabs,
    },
    props: {
@ -62,14 +31,10 @@
        required: true,
      },
    },
-    setup(props) {
-      const tabs = ['PROFILE', 'PREFERENCES']
-      const authUserPictureUrl: ComputedRef<string> = computed(() =>
-        props.user.picture
-          ? `${getApiUrl()}/users/${props.user.username}/picture?${Date.now()}`
-          : ''
-      )
-      return { authUserPictureUrl, tabs }
+    setup() {
+      return {
+        tabs: ['PROFILE', 'PREFERENCES'],
+      }
    },
  })
 </script>
@ -84,62 +49,5 @@
      width: 100%;
      margin: 0 auto 50px auto;
    }
-
-    .user-header {
-      display: flex;
-      align-items: stretch;
-
-      .user-details {
-        flex-grow: 1;
-        padding: $default-padding;
-        display: flex;
-        flex-direction: column;
-        align-items: center;
-
-        .user-name {
-          font-size: 2em;
-          height: 60%;
-        }
-
-        .user-stats {
-          display: flex;
-          gap: $default-padding * 4;
-          .user-stat {
-            display: flex;
-            flex-direction: column;
-            align-items: center;
-            padding-top: $default-padding;
-            .stat-number,
-            .stat-label {
-              padding: 0 $default-padding * 0.5;
-            }
-            .stat-number {
-              font-weight: bold;
-              font-size: 1.5em;
-            }
-          }
-        }
-
-        @media screen and (max-width: $x-small-limit) {
-          .user-name {
-            font-size: 1.5em;
-          }
-
-          .user-stats {
-            gap: $default-padding * 2;
-            .user-stat {
-              .stat-number {
-                font-weight: bold;
-                font-size: 1.2em;
-              }
-
-              &.hide-small {
-                display: none;
-              }
-            }
-          }
-        }
-      }
-    }
  }
 </style>
--- a/fittrackee_client/src/components/User/ProfileEdition/UserInfosEdition.vue
+++ b/fittrackee_client/src/components/User/ProfileEdition/UserInfosEdition.vue
@ -81,7 +81,7 @@
          <button class="confirm" type="submit">
            {{ $t('buttons.SUBMIT') }}
          </button>
-          <button class="cancel" @click.prevent="$router.go(-1)">
+          <button class="cancel" @click.prevent="$router.push('/profile')">
            {{ $t('buttons.CANCEL') }}
          </button>
          <button class="danger" @click.prevent="updateDisplayModal(true)">
--- a/fittrackee_client/src/components/User/ProfileEdition/UserPreferencesEdition.vue
+++ b/fittrackee_client/src/components/User/ProfileEdition/UserPreferencesEdition.vue
@ -39,7 +39,10 @@
          <button class="confirm" type="submit">
            {{ $t('buttons.SUBMIT') }}
          </button>
-          <button class="cancel" @click.prevent="$router.go(-1)">
+          <button
+            class="cancel"
+            @click.prevent="$router.push('/profile/preferences')"
+          >
            {{ $t('buttons.CANCEL') }}
          </button>
        </div>
--- a/fittrackee_client/src/components/User/ProfileEdition/index.vue
+++ b/fittrackee_client/src/components/User/ProfileEdition/index.vue
@ -1,7 +1,9 @@
 <template>
  <div id="user-profile-edition">
    <Card>
-      <template #title>{{ $t(`user.PROFILE.${tab}_EDITION`) }}</template>
+      <template #title>
+        {{ $t(`user.PROFILE.${tab}_EDITION`) }}
+      </template>
      <template #content>
        <UserProfileTabs
          :tabs="tabs"
@ -9,20 +11,15 @@
          :edition="true"
          :disabled="loading"
        />
-        <UserInfosEdition v-if="tab === 'PROFILE'" :user="user" />
-        <UserPreferencesEdition v-if="tab === 'PREFERENCES'" :user="user" />
-        <UserPictureEdition v-if="tab === 'PICTURE'" :user="user" />
+        <router-view :user="user"></router-view>
      </template>
    </Card>
  </div>
 </template>

 <script lang="ts">
-  import { PropType, computed, defineComponent, ref } from 'vue'
+  import { computed, defineComponent, PropType } from 'vue'

-  import UserInfosEdition from '@/components/User/ProfileEdition/UserInfosEdition.vue'
-  import UserPictureEdition from '@/components/User/ProfileEdition/UserPictureEdition.vue'
-  import UserPreferencesEdition from '@/components/User/ProfileEdition/UserPreferencesEdition.vue'
  import UserProfileTabs from '@/components/User/UserProfileTabs.vue'
  import { USER_STORE } from '@/store/constants'
  import { IAuthUserProfile } from '@/types/user'
@ -31,9 +28,6 @@
  export default defineComponent({
    name: 'ProfileEdition',
    components: {
-      UserInfosEdition,
-      UserPictureEdition,
-      UserPreferencesEdition,
      UserProfileTabs,
    },
    props: {
@ -46,14 +40,12 @@
        required: true,
      },
    },
-    setup(props) {
+    setup() {
      const store = useStore()
-      const tabs = ['PROFILE', 'PICTURE', 'PREFERENCES']
-      const selectedTab = ref(props.tab)
-      const loading = computed(
-        () => store.getters[USER_STORE.GETTERS.USER_LOADING]
-      )
-      return { loading, selectedTab, tabs }
+      return {
+        loading: computed(() => store.getters[USER_STORE.GETTERS.USER_LOADING]),
+        tabs: ['PROFILE', 'PICTURE', 'PREFERENCES'],
+      }
    },
  })
 </script>
--- a/fittrackee_client/src/router/index.ts
+++ b/fittrackee_client/src/router/index.ts
@ -1,18 +1,31 @@
 import { createRouter, createWebHistory, RouteRecordRaw } from 'vue-router'

+import Profile from '@/components/User/ProfileDisplay/index.vue'
+import UserInfos from '@/components/User/ProfileDisplay/UserInfos.vue'
+import UserPreferences from '@/components/User/ProfileDisplay/UserPreferences.vue'
+import ProfileEdition from '@/components/User/ProfileEdition/index.vue'
+import UserInfosEdition from '@/components/User/ProfileEdition/UserInfosEdition.vue'
+import UserPictureEdition from '@/components/User/ProfileEdition/UserPictureEdition.vue'
+import UserPreferencesEdition from '@/components/User/ProfileEdition/UserPreferencesEdition.vue'
 import store from '@/store'
 import { USER_STORE } from '@/store/constants'
-import AddWorkout from '@/views/AddWorkout.vue'
 import Dashboard from '@/views/DashBoard.vue'
 import LoginOrRegister from '@/views/LoginOrRegister.vue'
 import NotFoundView from '@/views/NotFoundView.vue'
 import PasswordResetView from '@/views/PasswordResetView.vue'
 import ProfileView from '@/views/ProfileView.vue'
 import StatisticsView from '@/views/StatisticsView.vue'
+import AddWorkout from '@/views/workouts/AddWorkout.vue'
 import EditWorkout from '@/views/workouts/EditWorkout.vue'
 import Workout from '@/views/workouts/Workout.vue'
 import Workouts from '@/views/workouts/WorkoutsView.vue'

+const getTabFromPath = (path: string): string => {
+  const regex = /(\/profile)(\/edit)*(\/*)/
+  const tag = path.replace(regex, '').toUpperCase()
+  return tag === '' ? 'PROFILE' : tag.toUpperCase()
+}
+
 const routes: Array<RouteRecordRaw> = [
  {
    path: '/',
@ -31,12 +44,6 @@ const routes: Array<RouteRecordRaw> = [
    component: LoginOrRegister,
    props: { action: 'register' },
  },
-  {
-    path: '/profile',
-    name: 'Profile',
-    component: ProfileView,
-    props: { edition: false, tab: 'PROFILE' },
-  },
  {
    path: '/password-reset/sent',
    name: 'PasswordEmailSent',
@ -62,28 +69,56 @@ const routes: Array<RouteRecordRaw> = [
    props: { action: 'reset' },
  },
  {
-    path: '/profile/edit/picture',
-    name: 'UserPictureEdition',
+    path: '/profile',
+    name: 'Profile',
    component: ProfileView,
-    props: { edition: true, tab: 'PICTURE' },
-  },
-  {
-    path: '/profile/preferences',
-    name: 'UserPreferences',
-    component: ProfileView,
-    props: { edition: false, tab: 'PREFERENCES' },
-  },
-  {
-    path: '/profile/edit/preferences',
-    name: 'UserPreferencesEdition',
-    component: ProfileView,
-    props: { edition: true, tab: 'PREFERENCES' },
-  },
-  {
-    path: '/profile/edit',
-    name: 'ProfileEdition',
-    component: ProfileView,
-    props: { edition: true, tab: 'PROFILE' },
+    children: [
+      {
+        path: '',
+        name: 'UserProfile',
+        component: Profile,
+        props: (route) => ({
+          tab: getTabFromPath(route.path),
+        }),
+        children: [
+          {
+            path: '',
+            name: 'UserInfos',
+            component: UserInfos,
+          },
+          {
+            path: 'preferences',
+            name: 'UserPreferences',
+            component: UserPreferences,
+          },
+        ],
+      },
+      {
+        path: 'edit',
+        name: 'UserProfileEdition',
+        component: ProfileEdition,
+        props: (route) => ({
+          tab: getTabFromPath(route.path),
+        }),
+        children: [
+          {
+            path: '',
+            name: 'UserInfosEdition',
+            component: UserInfosEdition,
+          },
+          {
+            path: 'picture',
+            name: 'UserPictureEdition',
+            component: UserPictureEdition,
+          },
+          {
+            path: 'preferences',
+            name: 'UserPreferencesEdition',
+            component: UserPreferencesEdition,
+          },
+        ],
+      },
+    ],
  },
  {
    path: '/statistics',
--- a/fittrackee_client/src/views/ProfileView.vue
+++ b/fittrackee_client/src/views/ProfileView.vue
@ -1,41 +1,24 @@
 <template>
  <div id="profile" class="container" v-if="authUser.username">
-    <ProfileEdition :user="authUser" :tab="tab" v-if="edition" />
-    <Profile :user="authUser" :tab="tab" v-else />
+    <router-view :user="authUser"></router-view>
  </div>
 </template>

 <script lang="ts">
-  import { computed, defineComponent } from 'vue'
+  import { computed, ComputedRef, defineComponent } from 'vue'

-  import Profile from '@/components/User/ProfileDisplay/index.vue'
-  import ProfileEdition from '@/components/User/ProfileEdition/index.vue'
  import { USER_STORE } from '@/store/constants'
+  import { IAuthUserProfile } from '@/types/user'
  import { useStore } from '@/use/useStore'

  export default defineComponent({
    name: 'ProfileView',
-    components: {
-      Profile,
-      ProfileEdition,
-    },
-    props: {
-      edition: {
-        type: Boolean,
-        required: true,
-      },
-      tab: {
-        type: String,
-        required: true,
-      },
-    },
    setup() {
      const store = useStore()
-      return {
-        authUser: computed(
-          () => store.getters[USER_STORE.GETTERS.AUTH_USER_PROFILE]
-        ),
-      }
+      const authUser: ComputedRef<IAuthUserProfile> = computed(
+        () => store.getters[USER_STORE.GETTERS.AUTH_USER_PROFILE]
+      )
+      return { authUser }
    },
  })
 </script>
--- a/fittrackee_client/src/views/workouts/AddWorkout.vue
+++ b/fittrackee_client/src/views/workouts/AddWorkout.vue