From 90d1e50b9ece4cb57a834624b08239da616b5461 Mon Sep 17 00:00:00 2001
From: Alina Beck <alina.beck@mail.com>
Date: Tue, 14 Jan 2020 18:18:50 +0300
Subject: [PATCH] refactor EmotionButton

---
 .../EmotionButton/EmotionButton.vue           | 55 +++++++++++++++++++
 webapp/components/Emotions/Emotions.vue       | 31 ++++++-----
 .../EmotionsButton/EmotionsButton.vue         | 55 -------------------
 webapp/pages/post/_id/_slug/index.vue         |  6 +-
 4 files changed, 73 insertions(+), 74 deletions(-)
 create mode 100644 webapp/components/EmotionButton/EmotionButton.vue
 delete mode 100644 webapp/components/EmotionsButton/EmotionsButton.vue

diff --git a/webapp/components/EmotionButton/EmotionButton.vue b/webapp/components/EmotionButton/EmotionButton.vue
new file mode 100644
index 000000000..97eabd7df
--- /dev/null
+++ b/webapp/components/EmotionButton/EmotionButton.vue
@@ -0,0 +1,55 @@
+<template>
+  <div class="emotion-button">
+    <base-button :id="emotion" circle ghost @click="$emit('toggleEmotion', emotion)">
+      <img :src="emojiPath" />
+    </base-button>
+    <label class="label" :for="emotion">{{ $t(`contribution.emotions-label.${emotion}`) }}</label>
+    <p v-if="count !== null" class="count">{{ emotionCount }}x</p>
+  </div>
+</template>
+
+<script>
+export default {
+  props: {
+    emojiPath: {
+      type: String,
+      required: true,
+    },
+    emotion: {
+      type: String,
+      required: true,
+    },
+    emotionCount: {
+      type: Number,
+      default: null,
+    },
+  },
+}
+</script>
+
+<style lang="scss">
+.emotion-button {
+  flex-grow: 1;
+  display: flex;
+  flex-direction: column;
+  align-items: center;
+
+  > .base-button {
+    padding: 0;
+
+    &:hover {
+      padding: $space-xxx-small;
+    }
+  }
+
+  > .label {
+    margin-top: $space-xx-small;
+    font-size: $font-size-small;
+    cursor: pointer;
+  }
+
+  > .count {
+    margin: $space-xx-small 0;
+  }
+}
+</style>
diff --git a/webapp/components/Emotions/Emotions.vue b/webapp/components/Emotions/Emotions.vue
index 5c65a9c2a..96389d10d 100644
--- a/webapp/components/Emotions/Emotions.vue
+++ b/webapp/components/Emotions/Emotions.vue
@@ -1,29 +1,26 @@
 <template>
-  <ds-flex :gutter="{ lg: 'large' }" class="emotions-flex">
-    <ds-flex-item
+  <div class="emotions-button-group">
+    <emotion-button
       v-for="emotion in Object.keys(PostsEmotionsCountByEmotion)"
       :key="emotion"
-      :width="{ lg: '100%' }"
-    >
-      <hc-emotions-button
-        @toggleEmotion="toggleEmotion"
-        :PostsEmotionsCountByEmotion="PostsEmotionsCountByEmotion"
-        :iconPath="iconPath(emotion)"
-        :emotion="emotion"
-      />
-    </ds-flex-item>
-  </ds-flex>
+      :emojiPath="iconPath(emotion)"
+      :emotion="emotion"
+      :emotionCount="PostsEmotionsCountByEmotion[emotion]"
+      @toggleEmotion="toggleEmotion"
+    />
+  </div>
 </template>
+
 <script>
 import gql from 'graphql-tag'
 import { mapGetters } from 'vuex'
-import HcEmotionsButton from '~/components/EmotionsButton/EmotionsButton'
+import EmotionButton from '~/components/EmotionButton/EmotionButton'
 import { PostsEmotionsByCurrentUser } from '~/graphql/PostQuery.js'
 import PostMutations from '~/graphql/PostMutations.js'
 
 export default {
   components: {
-    HcEmotionsButton,
+    EmotionButton,
   },
   props: {
     post: { type: Object, default: () => {} },
@@ -115,3 +112,9 @@ export default {
   },
 }
 </script>
+
+<style lang="scss">
+.emotions-button-group {
+  display: flex;
+}
+</style>
diff --git a/webapp/components/EmotionsButton/EmotionsButton.vue b/webapp/components/EmotionsButton/EmotionsButton.vue
deleted file mode 100644
index 04d3f7d96..000000000
--- a/webapp/components/EmotionsButton/EmotionsButton.vue
+++ /dev/null
@@ -1,55 +0,0 @@
-<template>
-  <div class="emotions-button">
-    <base-button circle ghost @click="toggleEmotion(emotion)">
-      <img :src="iconPath" />
-    </base-button>
-    <ds-space margin-bottom="xx-small" />
-    <div class="emotions-mobile-space">
-      <p class="emotions-label">{{ $t(`contribution.emotions-label.${emotion}`) }}</p>
-      <p style="display: inline" :key="PostsEmotionsCountByEmotion[emotion]">
-        {{ PostsEmotionsCountByEmotion[emotion] }}x
-      </p>
-    </div>
-  </div>
-</template>
-<script>
-export default {
-  props: {
-    iconPath: { type: String, default: null },
-    PostsEmotionsCountByEmotion: { type: Object, default: () => {} },
-    emotion: { type: String, default: null },
-  },
-  methods: {
-    toggleEmotion(emotion) {
-      this.$emit('toggleEmotion', emotion)
-    },
-  },
-}
-</script>
-<style lang="scss">
-.emotions-button {
-  > .base-button {
-    padding: 0;
-
-    &:hover {
-      padding: 2px;
-    }
-  }
-}
-
-.emotions-flex {
-  width: 100%;
-  justify-content: space-evenly;
-  text-align: center;
-}
-
-.emotions-label {
-  font-size: $font-size-small;
-}
-
-@media only screen and (max-width: 960px) {
-  .emotions-mobile-space {
-    margin-bottom: 32px;
-  }
-}
-</style>
diff --git a/webapp/pages/post/_id/_slug/index.vue b/webapp/pages/post/_id/_slug/index.vue
index e6b6eef44..1d7ddb310 100644
--- a/webapp/pages/post/_id/_slug/index.vue
+++ b/webapp/pages/post/_id/_slug/index.vue
@@ -47,13 +47,9 @@
       </div>
       <ds-space margin-top="x-large">
         <ds-flex :gutter="{ lg: 'small' }">
-          <ds-flex-item
-            :width="{ lg: '75%', md: '75%', sm: '75%', base: '100%' }"
-            class="emotions-buttons-mobile"
-          >
+          <ds-flex-item :width="{ lg: '75%', md: '75%', sm: '75%', base: '100%' }">
             <hc-emotions :post="post" />
           </ds-flex-item>
-          <ds-flex-item :width="{ lg: '10%', md: '3%', sm: '3%' }" />
           <!-- Shout Button -->
           <ds-flex-item
             :width="{ lg: '15%', md: '22%', sm: '22%', base: '100%' }"