From da80c858c866252e67d9f7157592664bb203ab59 Mon Sep 17 00:00:00 2001
From: Sara Boutej <40363501+SaraBoutej@users.noreply.github.com>
Date: Thu, 4 Jul 2024 14:42:48 +0100
Subject: [PATCH] feat: Sites Management Card review - MEED-7165 -
 Meeds-io/MIPs#137 (#152)

This PR allows to review the UI of the sites management card and drawer
---
 .../ManageAccessPermission.vue                |  6 +--
 .../ManageEditPermission.vue                  |  2 +-
 .../site-management/components/SiteCard.vue   |  2 +-
 .../components/SitePropertiesDrawer.vue       | 40 +++++++------------
 4 files changed, 19 insertions(+), 31 deletions(-)

diff --git a/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManageAccessPermission.vue b/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManageAccessPermission.vue
index af24c466a..0db931db7 100644
--- a/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManageAccessPermission.vue
+++ b/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManageAccessPermission.vue
@@ -18,14 +18,14 @@
 -->
 <template>
   <div>
-    <p class="font-weight-bold text-start text-color body-2">{{ whoCanView }}</p>
+    <p class="text-start">{{ whoCanView }}</p>
     <v-select
       v-model="type"
       :items="typeLabel"
       item-text="text"
       item-value="value"
       dense
-      class="caption"
+      class="pt-0"
       outlined />
     <exo-identity-suggester
       v-if="showAccessPermissionsGroupSuggester"
@@ -41,7 +41,7 @@
       include-groups
       include-spaces
       required />
-    <span v-if="!accessPermissions.length && showAccessPermissionsGroupSuggester" class="caption mx-2 mt-n3 position-absolute error-color">
+    <span v-if="!accessPermissions.length && showAccessPermissionsGroupSuggester" class="text-subtitle mx-2 mt-n3 position-absolute error-color">
       {{ $t('siteNavigation.required.error.message') }}
     </span>
     <template v-if="showAccessPermissionsList">
diff --git a/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManageEditPermission.vue b/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManageEditPermission.vue
index a64577a44..442fe2115 100644
--- a/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManageEditPermission.vue
+++ b/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManageEditPermission.vue
@@ -18,7 +18,7 @@
 -->
 <template>
   <div>
-    <p class="font-weight-bold text-start text-color body-2">{{ whoCanEdit }}</p>
+    <p class="text-start">{{ whoCanEdit }}</p>
     <exo-identity-suggester
       ref="navigationNodeEditPermissions"
       :labels="suggesterLabels"
diff --git a/layout-webapp/src/main/webapp/vue-app/site-management/components/SiteCard.vue b/layout-webapp/src/main/webapp/vue-app/site-management/components/SiteCard.vue
index 336f72d73..e8904581a 100644
--- a/layout-webapp/src/main/webapp/vue-app/site-management/components/SiteCard.vue
+++ b/layout-webapp/src/main/webapp/vue-app/site-management/components/SiteCard.vue
@@ -33,7 +33,7 @@
         :site="site" />
     </v-toolbar>
     <div class="d-flex flex-grow-0 flex-shrink-1 rich-editor-content">
-      <span class="align-self-center text-wrap text-left text-break mx-2 text-truncate-4" v-sanitized-html="site.description"></span>
+      <span class="align-self-center text-wrap text-left text-break mx-4 text-truncate-4" v-sanitized-html="site.description"></span>
     </div>
     <v-footer absolute class="px-0 py-0">
       <v-toolbar
diff --git a/layout-webapp/src/main/webapp/vue-app/site-management/components/SitePropertiesDrawer.vue b/layout-webapp/src/main/webapp/vue-app/site-management/components/SitePropertiesDrawer.vue
index f3f3d0046..6bd2d76a7 100644
--- a/layout-webapp/src/main/webapp/vue-app/site-management/components/SitePropertiesDrawer.vue
+++ b/layout-webapp/src/main/webapp/vue-app/site-management/components/SitePropertiesDrawer.vue
@@ -49,20 +49,16 @@
             required
             @blur="convertSiteName">
             <template #title>
-              <v-label>
-                <span class="text-color font-weight-bold">
-                  {{ $t('siteManagement.label.siteLabel.title') }} *              
-                </span>
-              </v-label>
+              <span>
+                {{ $t('siteManagement.label.siteLabel.title') }} *              
+              </span>
             </template>
           </translation-text-field>
         </v-card-text>
         <v-card-text class="d-flex pb-2">
-          <v-label>
-            <span class="text-color font-weight-bold">
-              {{ $t('siteManagement.label.siteName.title') }} *              
-            </span>
-          </v-label>
+          <span>
+            {{ $t('siteManagement.label.siteName.title') }} *              
+          </span>
         </v-card-text>
         <v-card-text class="d-flex py-0">
           <v-text-field
@@ -89,11 +85,9 @@
             back-icon
             rich-editor>
             <template #title>
-              <v-label>
-                <span class="text-color font-weight-bold">
-                  {{ $t('siteManagement.label.siteDescription.title') }}           
-                </span>
-              </v-label>
+              <span>
+                {{ $t('siteManagement.label.siteDescription.title') }}           
+              </span>
             </template>
             <rich-editor
               id="siteDescription"
@@ -105,12 +99,8 @@
           </translation-text-field>
         </v-card-text>
         <v-card-text class="mt-3 py-4 px-3">
-          <v-label>
-            <span class="text-color font-weight-bold mb-2 px-1 text-capitalize"> {{ $t('siteManagement.label.banner') }}</span>
-          </v-label>
-          <v-label>
-            <span class="text-color caption px-1 text-capitalize"> {{ $t('siteManagement.label.caption') }}</span>
-          </v-label>
+          <p class="mb-2 px-1 text-capitalize"> {{ $t('siteManagement.label.banner') }}</p>
+          <p class="text-subtitle  px-1 text-capitalize"> {{ $t('siteManagement.label.caption') }}</p>
           <site-management-banner-selector
             ref="siteBannerSelector"
             :src="siteBannerUrl"
@@ -121,17 +111,15 @@
             @reset="resetBannerUploadId" />
         </v-card-text>
         <v-card-text class="mt-4">
-          <v-label>
-            <span class="text-color font-weight-bold mb-2"> {{ $t('siteManagement.label.displayOrder') }}</span>
-          </v-label>
+          <span class="mb-2"> {{ $t('siteManagement.label.displayOrder') }}</span>
           <v-row class="mx-0 mt-2">
             <v-col class="d-flex flex-row px-0 py-0 col-10">
               <v-switch
                 v-model="displayed"
                 :disabled="displayedDisabled"
                 class="mt-2" />
-              <label v-if="displayed" class="subtitle-1 mx-1"> {{ $t('siteManagement.label.displayed') }} </label>
-              <label v-else class="subtitle-1 mx-1"> {{ $t('siteManagement.label.notDisplayed') }} </label>
+              <p v-if="displayed" class="mx-1"> {{ $t('siteManagement.label.displayed') }} </p>
+              <p v-else class="mx-1"> {{ $t('siteManagement.label.notDisplayed') }} </p>
             </v-col>
             <v-col class="col-2 px-0 py-0 mt-n1 orderDisplay">
               <v-text-field