From 3e5811988862d1bc80210e227913103e03e24a35 Mon Sep 17 00:00:00 2001
From: Sara Boutej <40363501+SaraBoutej@users.noreply.github.com>
Date: Mon, 8 Jul 2024 16:07:29 +0100
Subject: [PATCH] feat: Review UX to add node with external link - MEED-6835 -
 Meeds-io/MIPs#137 (#157)

---
 .../manage-permissions/ManageEditPermission.vue          | 2 +-
 .../manage-permissions/ManagePermissionItem.vue          | 2 +-
 .../manage-permissions/ManagePermissionsDrawer.vue       | 9 ++++++---
 .../site-navigation/SiteNavigationNodeDrawer.vue         | 4 ++--
 4 files changed, 10 insertions(+), 7 deletions(-)

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 442fe2115..88bbb2efb 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
@@ -32,7 +32,7 @@
       include-groups
       include-spaces
       required />
-    <span v-if="!permission.group" class="caption mt-n3 mx-2 position-absolute error-color">
+    <span v-if="!permission.group" class="text-subtitle mt-n3 mx-2 position-absolute error-color">
       {{ $t('siteNavigation.required.error.message') }}
     </span>
     <manage-permission-item
diff --git a/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManagePermissionItem.vue b/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManagePermissionItem.vue
index 249a9d182..67151c985 100644
--- a/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManagePermissionItem.vue
+++ b/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManagePermissionItem.vue
@@ -22,7 +22,7 @@
       size="19"
       class="fas fa-users me-3" />
     <span
-      class="d-flex text-truncate font-weight-bold text-start text-color body-2 mt-2">
+      class="d-flex text-truncate font-weight-bold text-start mt-2">
       {{ displayName }}
     </span>
     <v-spacer />
diff --git a/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManagePermissionsDrawer.vue b/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManagePermissionsDrawer.vue
index d1b884835..30f1a0447 100644
--- a/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManagePermissionsDrawer.vue
+++ b/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/manage-permissions/ManagePermissionsDrawer.vue
@@ -26,9 +26,12 @@
     @closed="close">
     <template slot="title">
       <div class="d-flex">
-        <i
-          class="uiIcon uiArrowBAckIcon clickable"
-          @click="close"></i>
+        <v-icon
+          size="16"
+          class="clickable"
+          @click="close">
+          fas fa-arrow-left
+        </v-icon>
         <span class="ms-2"> {{ drawerTitle }}</span>
       </div>
     </template>
diff --git a/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/site-navigation/SiteNavigationNodeDrawer.vue b/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/site-navigation/SiteNavigationNodeDrawer.vue
index ffefff534..df77c3106 100644
--- a/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/site-navigation/SiteNavigationNodeDrawer.vue
+++ b/layout-webapp/src/main/webapp/vue-app/common-layout-components/components/site-navigation/SiteNavigationNodeDrawer.vue
@@ -129,12 +129,12 @@
                     v-model="link"
                     :placeholder="$t('siteNavigation.label.enterUrl') "
                     :rules="linkRules"
-                    class="pt-0 mb-3"
+                    class="pt-0 mb-0 ms-8"
                     type="text"
                     required
                     outlined
                     dense />
-                  <div class="d-flex align-center justify-space-between flex-row pb-2">
+                  <div class="d-flex align-center justify-space-between flex-row pb-2 ms-8">
                     <span>
                       {{ $t('siteNavigation.label.openSameTab') }}
                     </span>