feat: Review UX to add node with external link - MEED-6835 - Meeds-io…

…/MIPs#137 (#157)
Meeds-io · Aug 23, 2024 · 4b0a98b · 4b0a98b
1 parent e585f46
commit 4b0a98b
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 7 deletions.
diff --git a/...p/vue-app/common-layout-components/components/manage-permissions/ManageEditPermission.vue b/...p/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/...p/vue-app/common-layout-components/components/manage-permissions/ManagePermissionItem.vue b/...p/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/...ue-app/common-layout-components/components/manage-permissions/ManagePermissionsDrawer.vue b/...ue-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/.../vue-app/common-layout-components/components/site-navigation/SiteNavigationNodeDrawer.vue b/.../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>