From 9b23a11c5beaf8475703c883b0952a3747ed65d3 Mon Sep 17 00:00:00 2001
From: Nithin David Thomas <webofnithin@gmail.com>
Date: Thu, 3 Sep 2020 14:11:07 +0530
Subject: [PATCH] chore: Improves styling for conversation header (#1167)

Co-authored-by: Pranav Raj S <pranav@thoughtwoot.com>
---
 .../assets/scss/_helper-classes.scss          |  8 +++
 .../dashboard/assets/scss/_mixins.scss        | 10 +---
 .../assets/scss/widgets/_conv-header.scss     | 16 ++++++
 .../conversation/ConversationHeader.vue       |  6 +--
 .../widgets/conversation/MoreActions.vue      | 51 +++++++++++--------
 .../conversation/contact/ContactInfo.vue      |  2 +-
 .../shared/assets/stylesheets/colors.scss     | 23 ++++++---
 7 files changed, 77 insertions(+), 39 deletions(-)

diff --git a/app/javascript/dashboard/assets/scss/_helper-classes.scss b/app/javascript/dashboard/assets/scss/_helper-classes.scss
index cbcc31e42..a1bfa1462 100644
--- a/app/javascript/dashboard/assets/scss/_helper-classes.scss
+++ b/app/javascript/dashboard/assets/scss/_helper-classes.scss
@@ -69,3 +69,11 @@ select {
     border-top-left-radius: 0 !important;
   }
 }
+
+.justify-space-between {
+  justify-content: space-between;
+}
+
+.w-100 {
+  width: 100%;
+}
diff --git a/app/javascript/dashboard/assets/scss/_mixins.scss b/app/javascript/dashboard/assets/scss/_mixins.scss
index 36a87a4f9..a20a32ffe 100644
--- a/app/javascript/dashboard/assets/scss/_mixins.scss
+++ b/app/javascript/dashboard/assets/scss/_mixins.scss
@@ -1,3 +1,4 @@
+@import '~dashboard/assets/scss/variables';
 @import '~widget/assets/scss/mixins';
 
 $spinner-before-border-color: rgba(255, 255, 255, 0.7);
@@ -235,12 +236,3 @@ $spinner-before-border-color: rgba(255, 255, 255, 0.7);
   text-overflow: ellipsis;
   white-space: nowrap;
 }
-
-
-.justify-space-between {
-  justify-content: space-between;
-}
-
-.w-100 {
-  width: 100%;
-}
diff --git a/app/javascript/dashboard/assets/scss/widgets/_conv-header.scss b/app/javascript/dashboard/assets/scss/widgets/_conv-header.scss
index 878e5a788..20919a73a 100644
--- a/app/javascript/dashboard/assets/scss/widgets/_conv-header.scss
+++ b/app/javascript/dashboard/assets/scss/widgets/_conv-header.scss
@@ -41,6 +41,10 @@
     @include flex;
     @include flex-align($x: center, $y: middle);
 
+    &.hide {
+      visibility: hidden;
+    }
+
     .user--name {
       @include margin(0);
       font-size: $font-size-medium;
@@ -83,3 +87,15 @@
     }
   }
 }
+
+
+.header-actions-wrap {
+  display: flex;
+  flex-direction: row;
+  flex-grow: 1;
+  justify-content: flex-end;
+
+  &.has-open-sidebar {
+    justify-content: flex-end;
+  }
+}
diff --git a/app/javascript/dashboard/components/widgets/conversation/ConversationHeader.vue b/app/javascript/dashboard/components/widgets/conversation/ConversationHeader.vue
index 65567ae26..10860a2bd 100644
--- a/app/javascript/dashboard/components/widgets/conversation/ConversationHeader.vue
+++ b/app/javascript/dashboard/components/widgets/conversation/ConversationHeader.vue
@@ -1,6 +1,6 @@
 <template>
   <div class="conv-header">
-    <div v-if="!isContactPanelOpen" class="user">
+    <div class="user" :class="{ hide: isContactPanelOpen }">
       <Thumbnail
         :src="currentContact.thumbnail"
         size="40px"
@@ -25,8 +25,8 @@
       </div>
     </div>
     <div
-      class="flex-container"
-      :class="{ 'justify-space-between w-100': isContactPanelOpen }"
+      class="header-actions-wrap"
+      :class="{ 'has-open-sidebar': isContactPanelOpen }"
     >
       <div class="multiselect-box ion-headphone">
         <multiselect
diff --git a/app/javascript/dashboard/components/widgets/conversation/MoreActions.vue b/app/javascript/dashboard/components/widgets/conversation/MoreActions.vue
index a5f9860ce..af7217d09 100644
--- a/app/javascript/dashboard/components/widgets/conversation/MoreActions.vue
+++ b/app/javascript/dashboard/components/widgets/conversation/MoreActions.vue
@@ -5,9 +5,8 @@
       :status="currentChat.status"
     />
     <woot-button
-      class="success more--button"
-      icon="ion-more"
-      :class="buttonClass"
+      class="clear more--button"
+      icon="ion-android-more-vertical"
       @click="toggleConversationActions"
     />
     <div
@@ -18,17 +17,15 @@
     >
       <button
         v-if="!currentChat.muted"
-        class="button small clear row nice alert small-6 action--button"
+        class="button small clear row alert small-6 action--button"
         @click="mute"
       >
-        <i class="icon ion-volume-mute" />
         <span>{{ $t('CONTACT_PANEL.MUTE_CONTACT') }}</span>
       </button>
       <button
-        class="button small clear row nice small-6 action--button"
+        class="button small clear row small-6 action--button"
         @click="toggleEmailActionsModal"
       >
-        <i class="icon ion-ios-copy" />
         {{ $t('CONTACT_PANEL.SEND_TRANSCRIPT') }}
       </button>
     </div>
@@ -46,7 +43,6 @@ import { mixin as clickaway } from 'vue-clickaway';
 import alertMixin from 'shared/mixins/alertMixin';
 import EmailTranscriptModal from './EmailTranscriptModal';
 import ResolveAction from '../../buttons/ResolveAction';
-import wootConstants from '../../../constants';
 
 export default {
   components: {
@@ -64,12 +60,6 @@ export default {
     ...mapGetters({
       currentChat: 'getSelectedChat',
     }),
-
-    buttonClass() {
-      return this.currentChat.status !== wootConstants.STATUS_TYPE.OPEN
-        ? 'warning'
-        : 'success';
-    },
   },
   methods: {
     mute() {
@@ -91,11 +81,17 @@ export default {
 };
 </script>
 <style scoped lang="scss">
+@import '~dashboard/assets/scss/mixins';
+
 .more--button {
   align-items: center;
   display: flex;
-  margin-left: var(--space-smaller);
+  margin-left: var(--space-small);
   padding: var(--space-small);
+
+  &.clear.more--button {
+    color: var(--color-body);
+  }
 }
 
 .actions--container {
@@ -103,11 +99,25 @@ export default {
 }
 
 .dropdown-pane {
-  right: 0;
+  @include elegant-card;
+  @include border-light;
+  right: -12px;
   top: 48px;
-  border: 1px solid var(--s-100);
-  border-radius: var(--space-smaller);
   width: auto;
+
+  &::before {
+    @include arrow(top, var(--color-border-light), 14px);
+    top: -14px;
+    position: absolute;
+    right: 6px;
+  }
+
+  &::after {
+    @include arrow(top, white, var(--space-slab));
+    top: -12px;
+    position: absolute;
+    right: var(--space-small);
+  }
 }
 
 .dropdown-pane--open {
@@ -119,10 +129,11 @@ export default {
   display: flex;
   align-items: center;
   width: 100%;
-  padding: var(--space-small) 0;
+  padding: var(--space-small) var(--space-smaller);
+  font-size: var(--font-size-small);
 
   .icon {
-    margin-right: var(--space-small);
+    margin-right: var(--space-smaller);
     min-width: var(--space-normal);
   }
 }
diff --git a/app/javascript/dashboard/routes/dashboard/conversation/contact/ContactInfo.vue b/app/javascript/dashboard/routes/dashboard/conversation/contact/ContactInfo.vue
index 90ff44543..b99000e67 100644
--- a/app/javascript/dashboard/routes/dashboard/conversation/contact/ContactInfo.vue
+++ b/app/javascript/dashboard/routes/dashboard/conversation/contact/ContactInfo.vue
@@ -3,7 +3,7 @@
     <div class="contact--info">
       <thumbnail
         :src="contact.thumbnail"
-        size="48px"
+        size="64px"
         :badge="channelType"
         :username="contact.name"
         :status="contact.availability_status"
diff --git a/app/javascript/shared/assets/stylesheets/colors.scss b/app/javascript/shared/assets/stylesheets/colors.scss
index 40d05776e..94c57973a 100644
--- a/app/javascript/shared/assets/stylesheets/colors.scss
+++ b/app/javascript/shared/assets/stylesheets/colors.scss
@@ -1,7 +1,7 @@
 :root {
   --white: #fff;
 
-  --w-50:  #E3F2FF;
+  --w-50: #E3F2FF;
   --w-100: #BBDDFF;
   --w-200: #8FC9FF;
   --w-300: #61B3FF;
@@ -12,7 +12,7 @@
   --w-800: #2161CA;
   --w-900: #1F41AB;
 
-  --g-50:  #E6F8E6;
+  --g-50: #E6F8E6;
   --g-100: #C4EEC2;
   --g-200: #9DE29A;
   --g-300: #6FD86F;
@@ -23,7 +23,7 @@
   --g-800: #009000;
   --g-900: #007000;
 
-  --y-50:  #FFFEE8;
+  --y-50: #FFFEE8;
   --y-100: #FFFAC5;
   --y-200: #FFF69E;
   --y-300: #FEF176;
@@ -34,7 +34,7 @@
   --y-800: #FDAD2A;
   --y-900: #F9841B;
 
-  --s-50:  #E7EEFB;
+  --s-50: #E7EEFB;
   --s-100: #C8D6E6;
   --s-200: #ABBACE;
   --s-300: #8C9EB6;
@@ -45,7 +45,7 @@
   --s-800: #314155;
   --s-900: #1F2D3D;
 
-  --b-50:#F8F9FE;
+  --b-50: #F8F9FE;
   --b-100: #F2F3F7;
   --b-200: #E9EAEF;
   --b-300: #DADBDF;
@@ -56,7 +56,7 @@
   --b-800: #3C3D40;
   --b-900: #1B1C1F;
 
-  --r-50:  #FFEBEE;
+  --r-50: #FFEBEE;
   --r-100: #FFCCD1;
   --r-200: #F69898;
   --r-300: #EF6F6F;
@@ -66,4 +66,15 @@
   --r-700: #DE1E27;
   --r-800: #D11320;
   --r-900: #C30011;
+
+  // Common color aliases
+  --color-heading: #1f2d3d;
+  --color-body: #3c4858;
+
+  --color-border: #e0e6ed;
+  --color-border-light: #f0f4f5;
+  --color-border-dark: #cad0d4;
+
+  --color-background: #f4f6fb;
+  --color-background-light: #f9fafc;
 }