From 14c1a0c55893c353b5709d606da59ca19cf80e48 Mon Sep 17 00:00:00 2001
From: Pranav C <pranavxc@gmail.com>
Date: Tue, 5 Oct 2021 18:37:52 +0530
Subject: [PATCH] feat: add feedback form close option

Signed-off-by: Pranav C <pranavxc@gmail.com>
---
 packages/nc-gui/components/feedbackForm.vue   | 52 +++++++++++++++----
 packages/nc-gui/components/utils/language.vue | 14 +++--
 packages/nc-gui/store/windows.js              |  6 ++-
 3 files changed, 57 insertions(+), 15 deletions(-)

diff --git a/packages/nc-gui/components/feedbackForm.vue b/packages/nc-gui/components/feedbackForm.vue
index 70671028cd..d154984028 100644
--- a/packages/nc-gui/components/feedbackForm.vue
+++ b/packages/nc-gui/components/feedbackForm.vue
@@ -1,21 +1,51 @@
 <template>
-  <iframe
-    src="https://docs.google.com/forms/d/e/1FAIpQLSeTlAfZjszgr53lArz3NvUEnJGOT9JtG9NAU5d0oQwunDS2Pw/viewform?embedded=true"
-    width="100%"
-    height="500"
-    frameborder="0"
-    marginheight="0"
-    marginwidth="0"
-  >Loading…
-  </iframe>
+  <div v-if="!feedbackFormHidden" class="nc-feedback-form-wrapper">
+    <v-icon class="nc-close-icon" large @click="feedbackFormHidden = true">
+      mdi-close-circle-outline
+    </v-icon>
+
+    <iframe
+      src="https://docs.google.com/forms/d/e/1FAIpQLSeTlAfZjszgr53lArz3NvUEnJGOT9JtG9NAU5d0oQwunDS2Pw/viewform?embedded=true"
+      width="100%"
+      height="500"
+      frameborder="0"
+      marginheight="0"
+      marginwidth="0"
+    >Loading…
+    </iframe>
+  </div>
+  <div v-else />
 </template>
 
 <script>
 export default {
-  name: 'FeedbackForm'
+  name: 'FeedbackForm',
+  computed: {
+    feedbackFormHidden: {
+      get() {
+        return this.$store.state.windows.feedbackFormHidden
+      },
+      set(val) {
+        this.$store.commit('windows/MutFeedbackFormHidden', val)
+      }
+    }
+  }
 }
 </script>
 
-<style scoped>
+<style scoped lang="scss">
+.nc-feedback-form-wrapper {
+  width: 100%;
+  position: relative;
+
+  iframe {
+    margin: 0 auto;
+  }
 
+  .nc-close-icon {
+    position: absolute;
+    top: 5px;
+    right: 10px;
+  }
+}
 </style>
diff --git a/packages/nc-gui/components/utils/language.vue b/packages/nc-gui/components/utils/language.vue
index 712ecd9457..2b337eb55b 100644
--- a/packages/nc-gui/components/utils/language.vue
+++ b/packages/nc-gui/components/utils/language.vue
@@ -6,7 +6,7 @@
           mdi-translate
         </v-icon>
       </template>
-      <v-list dense>
+      <v-list dense class="nc-language-list">
         <v-list-item-group
           v-model="language"
         >
@@ -116,6 +116,14 @@ export default {
 }
 </script>
 
-<style scoped>
-
+<style scoped lang="scss">
+::v-deep {
+  .nc-language-list {
+    max-height: 90vh;
+    overflow: auto;
+     .v-list-item{
+      min-height: 30px !important;
+    }
+  }
+}
 </style>
diff --git a/packages/nc-gui/store/windows.js b/packages/nc-gui/store/windows.js
index 4e0eeefe69..d229474f98 100644
--- a/packages/nc-gui/store/windows.js
+++ b/packages/nc-gui/store/windows.js
@@ -38,7 +38,8 @@ export const state = () => ({
   screensaver: true,
   autoApplyFilter: true,
   apiLoading: false,
-  includeM2M: false
+  includeM2M: false,
+  feedbackFormHidden: false
 })
 
 export const mutations = {
@@ -51,6 +52,9 @@ export const mutations = {
   MutToggleLogWindow(state, show) {
     state.logWindow = !state.logWindow
   },
+  MutFeedbackFormHidden(state, show) {
+    state.feedbackFormHidden = show
+  },
   MutScreensaver(state, show) {
     state.screensaver = show
   },