From 29a3c621c23e4ff94811c5df7ac1b99f27d86fba Mon Sep 17 00:00:00 2001
From: Arman <arman@appwrite.io>
Date: Mon, 15 Jan 2024 17:33:27 +0100
Subject: [PATCH 01/11] chore: enable nps for design review

---
 src/lib/components/feedback/feedback.svelte | 4 ++--
 src/lib/stores/feedback.ts                  | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/lib/components/feedback/feedback.svelte b/src/lib/components/feedback/feedback.svelte
index 98cb759bdb..da586e849b 100644
--- a/src/lib/components/feedback/feedback.svelte
+++ b/src/lib/components/feedback/feedback.svelte
@@ -40,8 +40,8 @@
             type="button"
             class="button is-text is-only-icon u-margin-inline-start-auto"
             style="--button-size:1.5rem;"
-            aria-label="Close Modal"
-            title="Close Modal"
+            aria-label="Close feedback"
+            title="Close feedback"
             on:click={() => feedback.toggleFeedback()}>
             <span class="icon-x" aria-hidden="true" />
         </button>
diff --git a/src/lib/stores/feedback.ts b/src/lib/stores/feedback.ts
index 86bf6ccae3..67a8de4ebe 100644
--- a/src/lib/stores/feedback.ts
+++ b/src/lib/stores/feedback.ts
@@ -74,7 +74,7 @@ function createFeedbackStore() {
         elapsed: browser ? parseInt(localStorage.getItem('feedbackElapsed')) : 0,
         visualized: browser ? parseInt(localStorage.getItem('feedbackVisualized')) : 0,
         notification: false,
-        type: 'general',
+        type: 'nps', //TODO: change to general before release
         show: false
     });
     return {

From 2830334b280ccc53a64f7a368710dc557c789618 Mon Sep 17 00:00:00 2001
From: Arman <arman@appwrite.io>
Date: Wed, 17 Jan 2024 16:57:28 +0100
Subject: [PATCH 02/11] fix: feedback design

---
 src/lib/components/feedback/feedback.svelte     |  4 ++--
 src/lib/components/feedback/feedbackNPS.svelte  | 14 +++++++-------
 src/lib/stores/feedback.ts                      |  6 +++---
 src/routes/console/wizard/feedback/step1.svelte |  2 +-
 4 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/src/lib/components/feedback/feedback.svelte b/src/lib/components/feedback/feedback.svelte
index da586e849b..6124fd2997 100644
--- a/src/lib/components/feedback/feedback.svelte
+++ b/src/lib/components/feedback/feedback.svelte
@@ -20,7 +20,7 @@
             );
             addNotification({
                 type: 'success',
-                message: 'Feedback submitted successfully'
+                message: 'Thank you for your feedback'
             });
             feedback.toggleFeedback();
             feedbackData.reset();
@@ -33,7 +33,7 @@
     }
 </script>
 
-<section class="drop-section">
+<section class="drop-section u-padding-24">
     <header class="u-flex u-main-space-between u-gap-16">
         <h4 class="body-text-1 u-bold">{$selectedFeedback.title}</h4>
         <button
diff --git a/src/lib/components/feedback/feedbackNPS.svelte b/src/lib/components/feedback/feedbackNPS.svelte
index 613e10514a..69c37f6e50 100644
--- a/src/lib/components/feedback/feedbackNPS.svelte
+++ b/src/lib/components/feedback/feedbackNPS.svelte
@@ -4,11 +4,11 @@
     import Evaluation from './evaluation.svelte';
 </script>
 
-<Evaluation bind:value={$feedbackData.value}>
-    How likely are you to recommend Appwrite to a friend or colleague?
-</Evaluation>
-{#if $feedbackData.value}
-    <FormList>
+<FormList>
+    <Evaluation bind:value={$feedbackData.value}>
+        How likely are you to recommend Appwrite to a friend or colleague?
+    </Evaluation>
+    {#if $feedbackData.value !== null}
         <InputTextarea
             id="feedback"
             placeholder="Your message here"
@@ -26,5 +26,5 @@
             id="email"
             bind:value={$feedbackData.email}
             placeholder="Enter email" />
-    </FormList>
-{/if}
+    {/if}
+</FormList>
diff --git a/src/lib/stores/feedback.ts b/src/lib/stores/feedback.ts
index 67a8de4ebe..66646edf99 100644
--- a/src/lib/stores/feedback.ts
+++ b/src/lib/stores/feedback.ts
@@ -30,14 +30,14 @@ export type FeedbackOption = {
 export const feedbackOptions: FeedbackOption[] = [
     {
         type: 'general',
-        title: 'How can we improve?',
+        title: 'Your feedback is important to us',
         desc: 'Your feedback is important to us. Please be honest and tell us what you think.',
         component: FeedbackGeneral
     },
     {
         type: 'nps',
-        title: 'How are we doing?',
-        desc: "At Appwrite, we value the feedback of our users. That means you! We'd love to hear what you think.",
+        title: 'Your feedback is important to us',
+        desc: 'Please be honest and tell us what you think. If you would like to be contacted regarding your feedback, please share your details below.',
         component: FeedbackNps
     }
 ];
diff --git a/src/routes/console/wizard/feedback/step1.svelte b/src/routes/console/wizard/feedback/step1.svelte
index c3a6655903..c767a19db1 100644
--- a/src/routes/console/wizard/feedback/step1.svelte
+++ b/src/routes/console/wizard/feedback/step1.svelte
@@ -20,7 +20,7 @@
             );
             addNotification({
                 type: 'success',
-                message: 'Feedback submitted successfully'
+                message: 'Thank you for your feedback'
             });
         } catch (error) {
             addNotification({

From afd1b745b664d100ef9affa1cf218b42d9c1b1a1 Mon Sep 17 00:00:00 2001
From: Arman <arman@appwrite.io>
Date: Thu, 18 Jan 2024 14:47:22 +0100
Subject: [PATCH 03/11] fix:design review

---
 src/lib/components/feedback/evaluation.svelte | 4 ++--
 src/lib/components/feedback/feedback.svelte   | 2 +-
 src/lib/stores/feedback.ts                    | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/lib/components/feedback/evaluation.svelte b/src/lib/components/feedback/evaluation.svelte
index 96956714c4..3577ce9388 100644
--- a/src/lib/components/feedback/evaluation.svelte
+++ b/src/lib/components/feedback/evaluation.svelte
@@ -20,7 +20,7 @@
         {/each}
     </ul>
     <div class="u-flex u-main-space-between u-margin-block-start-8">
-        <span>Not at all likely</span>
-        <span>Extremely likely</span>
+        <span class:u-color-text-gray={value !== null}>Not at all likely</span>
+        <span class:u-color-text-gray={value !== null}>Extremely likely</span>
     </div>
 </fieldset>
diff --git a/src/lib/components/feedback/feedback.svelte b/src/lib/components/feedback/feedback.svelte
index 6124fd2997..db063755a7 100644
--- a/src/lib/components/feedback/feedback.svelte
+++ b/src/lib/components/feedback/feedback.svelte
@@ -35,7 +35,7 @@
 
 <section class="drop-section u-padding-24">
     <header class="u-flex u-main-space-between u-gap-16">
-        <h4 class="body-text-1 u-bold">{$selectedFeedback.title}</h4>
+        <h4 class="label u-bold" style:font-size="var(--font-size-1)">{$selectedFeedback.title}</h4>
         <button
             type="button"
             class="button is-text is-only-icon u-margin-inline-start-auto"
diff --git a/src/lib/stores/feedback.ts b/src/lib/stores/feedback.ts
index 66646edf99..f811617466 100644
--- a/src/lib/stores/feedback.ts
+++ b/src/lib/stores/feedback.ts
@@ -49,7 +49,7 @@ function createFeedbackDataStore() {
         message: '',
         name: '',
         email: '',
-        value: 0
+        value: null
     });
     return {
         set,
@@ -60,7 +60,7 @@ function createFeedbackDataStore() {
                 feedbackData.message = '';
                 feedbackData.name = '';
                 feedbackData.email = '';
-                feedbackData.value = 0;
+                feedbackData.value = null;
                 return feedbackData;
             });
         }

From d874c1d485e4019826b5a6a29b9e1b05db679396 Mon Sep 17 00:00:00 2001
From: Arman <arman@appwrite.io>
Date: Thu, 18 Jan 2024 15:06:57 +0100
Subject: [PATCH 04/11] fix: initialize localStorage

---
 src/lib/stores/feedback.ts        | 4 ++--
 src/routes/console/+layout.svelte | 3 +++
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/lib/stores/feedback.ts b/src/lib/stores/feedback.ts
index f811617466..325c4005dc 100644
--- a/src/lib/stores/feedback.ts
+++ b/src/lib/stores/feedback.ts
@@ -71,8 +71,8 @@ export const feedbackData = createFeedbackDataStore();
 
 function createFeedbackStore() {
     const { subscribe, update } = writable<Feedback>({
-        elapsed: browser ? parseInt(localStorage.getItem('feedbackElapsed')) : 0,
-        visualized: browser ? parseInt(localStorage.getItem('feedbackVisualized')) : 0,
+        elapsed: browser ? parseInt(localStorage.getItem('feedbackElapsed')) ?? 0 : 0,
+        visualized: browser ? parseInt(localStorage.getItem('feedbackVisualized')) ?? 0 : 0,
         notification: false,
         type: 'nps', //TODO: change to general before release
         show: false
diff --git a/src/routes/console/+layout.svelte b/src/routes/console/+layout.svelte
index e6cdd038c1..7a5aba0964 100644
--- a/src/routes/console/+layout.svelte
+++ b/src/routes/console/+layout.svelte
@@ -255,6 +255,9 @@
             }
         }
 
+        if (!localStorage.getItem('feedbackElapsed')) {
+            localStorage.setItem('feedbackElapsed', '0');
+        }
         setInterval(() => {
             checkForFeedback(INTERVAL);
         }, INTERVAL);

From 32fb608429a83604e72d9d47af16d7c7c50a81ef Mon Sep 17 00:00:00 2001
From: Arman <arman@appwrite.io>
Date: Tue, 23 Jan 2024 15:20:21 +0100
Subject: [PATCH 05/11] chore: upgrade package

---
 package-lock.json | 18 +++++++++---------
 package.json      |  4 ++--
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index bc90c96d99..1f016d1a0b 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -7,8 +7,8 @@
             "name": "@appwrite/console",
             "dependencies": {
                 "@appwrite.io/console": "^0.5.0",
-                "@appwrite.io/pink": "0.2.0",
-                "@appwrite.io/pink-icons": "0.2.0",
+                "@appwrite.io/pink": "0.3.0",
+                "@appwrite.io/pink-icons": "0.3.0",
                 "@popperjs/core": "^2.11.8",
                 "@sentry/svelte": "^7.66.0",
                 "@sentry/tracing": "^7.66.0",
@@ -168,19 +168,19 @@
             }
         },
         "node_modules/@appwrite.io/pink": {
-            "version": "0.2.0",
-            "resolved": "https://registry.npmjs.org/@appwrite.io/pink/-/pink-0.2.0.tgz",
-            "integrity": "sha512-ni6O/yPiWkCYQ68PNcbmus6QrPpIEl7xmzKHU2dkV2L19RVio8thtBtT9ckJJ8uBMGsczASWs0fUqOxIvBmtNw==",
+            "version": "0.3.0",
+            "resolved": "https://registry.npmjs.org/@appwrite.io/pink/-/pink-0.3.0.tgz",
+            "integrity": "sha512-VTYCfcEGnqScwGta7ort/qoKN0Gnyq2h/ROYcp9meEFSCjYBWMgWZWbwSSXsL8vwwxnN9aTh1sO5mJX0zHAPVg==",
             "dependencies": {
-                "@appwrite.io/pink-icons": "0.2.0",
+                "@appwrite.io/pink-icons": "0.3.0",
                 "normalize.css": "^8.0.1",
                 "the-new-css-reset": "^1.11.2"
             }
         },
         "node_modules/@appwrite.io/pink-icons": {
-            "version": "0.2.0",
-            "resolved": "https://registry.npmjs.org/@appwrite.io/pink-icons/-/pink-icons-0.2.0.tgz",
-            "integrity": "sha512-wThBUGdvpwT0q2fcUapiugfBiB4RFDvTlW6p+U7Dol61mmCpNteFSIoBPiPJzN5015n1v4UEWL/rdOia4CWdBQ=="
+            "version": "0.3.0",
+            "resolved": "https://registry.npmjs.org/@appwrite.io/pink-icons/-/pink-icons-0.3.0.tgz",
+            "integrity": "sha512-Jus3jsWMidne02/mqXbRtLFsuiYQiB3fjZ2ip8auqwhvk65yzGtodPiTPA8Us1fD2vaRlNx/jER5MK0XmqXKxg=="
         },
         "node_modules/@babel/code-frame": {
             "version": "7.23.5",
diff --git a/package.json b/package.json
index e84456270e..a46c0c28a8 100644
--- a/package.json
+++ b/package.json
@@ -19,8 +19,8 @@
     },
     "dependencies": {
         "@appwrite.io/console": "^0.5.0",
-        "@appwrite.io/pink": "0.2.0",
-        "@appwrite.io/pink-icons": "0.2.0",
+        "@appwrite.io/pink": "0.3.0",
+        "@appwrite.io/pink-icons": "0.3.0",
         "@popperjs/core": "^2.11.8",
         "@sentry/svelte": "^7.66.0",
         "@sentry/tracing": "^7.66.0",

From bd431a475272191da128e2cad8a51e90cd45ad81 Mon Sep 17 00:00:00 2001
From: Arman <arman@appwrite.io>
Date: Wed, 24 Jan 2024 14:48:26 +0100
Subject: [PATCH 06/11] feat: add notification

---
 src/lib/layout/header.svelte | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/lib/layout/header.svelte b/src/lib/layout/header.svelte
index da3aa7e7b9..81822850c4 100644
--- a/src/lib/layout/header.svelte
+++ b/src/lib/layout/header.svelte
@@ -133,6 +133,12 @@
         {/if}
         <DropList show={$feedback.show} scrollable on:blur={toggleFeedback}>
             <button class="button is-small is-text" on:click={toggleFeedback}>
+                <!-- TODO: invert this before release -->
+                {#if !$feedback.notification}
+                    <span
+                        class="notification u-position-absolute u-inset-block-start-8 u-inset-inline-end-8"
+                    ></span>
+                {/if}
                 <span class="text">Feedback</span>
             </button>
             <svelte:fragment slot="other">

From 83a725688c25a1e9c12176455e2749f009e10c2d Mon Sep 17 00:00:00 2001
From: Arman <arman@appwrite.io>
Date: Thu, 25 Jan 2024 17:41:28 +0100
Subject: [PATCH 07/11] fix: design

---
 src/lib/components/feedback/evaluation.svelte | 4 ++--
 src/lib/stores/feedback.ts                    | 8 ++++----
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/lib/components/feedback/evaluation.svelte b/src/lib/components/feedback/evaluation.svelte
index 3577ce9388..f80ad5535e 100644
--- a/src/lib/components/feedback/evaluation.svelte
+++ b/src/lib/components/feedback/evaluation.svelte
@@ -20,7 +20,7 @@
         {/each}
     </ul>
     <div class="u-flex u-main-space-between u-margin-block-start-8">
-        <span class:u-color-text-gray={value !== null}>Not at all likely</span>
-        <span class:u-color-text-gray={value !== null}>Extremely likely</span>
+        <span class:u-color-text-gray={value === null}>Not at all likely</span>
+        <span class:u-color-text-gray={value === null}>Extremely likely</span>
     </div>
 </fieldset>
diff --git a/src/lib/stores/feedback.ts b/src/lib/stores/feedback.ts
index 325c4005dc..e9aa5f30ae 100644
--- a/src/lib/stores/feedback.ts
+++ b/src/lib/stores/feedback.ts
@@ -30,14 +30,14 @@ export type FeedbackOption = {
 export const feedbackOptions: FeedbackOption[] = [
     {
         type: 'general',
-        title: 'Your feedback is important to us',
-        desc: 'Your feedback is important to us. Please be honest and tell us what you think.',
+        title: 'Help us improve Appwrite',
+        desc: 'Appwrite evolves with your input. Share your thoughts and help us improve Appwrite.',
         component: FeedbackGeneral
     },
     {
         type: 'nps',
-        title: 'Your feedback is important to us',
-        desc: 'Please be honest and tell us what you think. If you would like to be contacted regarding your feedback, please share your details below.',
+        title: 'Help us improve Appwrite',
+        desc: 'Appwrite evolves with your input. Share your thoughts and help us improve Appwrite. If you would like to be contacted regarding your feedback, please share your contact details below.',
         component: FeedbackNps
     }
 ];

From 370c344709d878e2e8ac097bc42da994a12fc653 Mon Sep 17 00:00:00 2001
From: Arman <arman@appwrite.io>
Date: Wed, 21 Feb 2024 14:07:37 +0100
Subject: [PATCH 08/11] fix: scroll on mobile

---
 src/lib/components/feedback/evaluation.svelte | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/lib/components/feedback/evaluation.svelte b/src/lib/components/feedback/evaluation.svelte
index f80ad5535e..7e19658466 100644
--- a/src/lib/components/feedback/evaluation.svelte
+++ b/src/lib/components/feedback/evaluation.svelte
@@ -2,11 +2,11 @@
     export let value: number = null;
 </script>
 
-<fieldset class="u-margin-block-start-8">
+<fieldset class="u-margin-block-start-8 u-min-width-0">
     <legend class="label is-required u-margin-0 u-line-height-1-5 u-bold">
         <slot />
     </legend>
-    <ul class="u-flex u-main-space-between u-margin-block-start-16">
+    <ol class="u-flex u-main-space-between u-margin-block-start-16 u-overflow-x-auto">
         {#each Array(11) as _, i}
             <li>
                 <input
@@ -18,7 +18,7 @@
                     checked={value === i} />
             </li>
         {/each}
-    </ul>
+    </ol>
     <div class="u-flex u-main-space-between u-margin-block-start-8">
         <span class:u-color-text-gray={value === null}>Not at all likely</span>
         <span class:u-color-text-gray={value === null}>Extremely likely</span>

From 53d756faaa4cb88de37f5ff56048b270a5f38f1a Mon Sep 17 00:00:00 2001
From: Arman <arman@appwrite.io>
Date: Tue, 19 Mar 2024 15:11:30 +0100
Subject: [PATCH 09/11] fix: hover

---
 src/lib/components/feedback/evaluation.svelte | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/lib/components/feedback/evaluation.svelte b/src/lib/components/feedback/evaluation.svelte
index 7e19658466..1e770e900b 100644
--- a/src/lib/components/feedback/evaluation.svelte
+++ b/src/lib/components/feedback/evaluation.svelte
@@ -6,7 +6,9 @@
     <legend class="label is-required u-margin-0 u-line-height-1-5 u-bold">
         <slot />
     </legend>
-    <ol class="u-flex u-main-space-between u-margin-block-start-16 u-overflow-x-auto">
+    <ol
+        class="u-flex u-main-space-between u-margin-block-start-16 u-overflow-x-auto"
+        style="padding-block: 0.13rem">
         {#each Array(11) as _, i}
             <li>
                 <input

From 1f9f6b0d6e09c6edf39cd75e1b2b6c0c1fdf7dd4 Mon Sep 17 00:00:00 2001
From: Arman <arman@appwrite.io>
Date: Mon, 3 Jun 2024 16:13:21 +0200
Subject: [PATCH 10/11] fix: remove forced nps

---
 src/lib/stores/feedback.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/stores/feedback.ts b/src/lib/stores/feedback.ts
index e9aa5f30ae..7f9b6b2004 100644
--- a/src/lib/stores/feedback.ts
+++ b/src/lib/stores/feedback.ts
@@ -74,7 +74,7 @@ function createFeedbackStore() {
         elapsed: browser ? parseInt(localStorage.getItem('feedbackElapsed')) ?? 0 : 0,
         visualized: browser ? parseInt(localStorage.getItem('feedbackVisualized')) ?? 0 : 0,
         notification: false,
-        type: 'nps', //TODO: change to general before release
+        type: 'general',
         show: false
     });
     return {

From 04434cb5e4887d92a6e5d745b1e29d3414842ee2 Mon Sep 17 00:00:00 2001
From: Arman <arman@appwrite.io>
Date: Mon, 3 Jun 2024 16:17:03 +0200
Subject: [PATCH 11/11] chore: remove forced notification

---
 src/lib/layout/header.svelte | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/lib/layout/header.svelte b/src/lib/layout/header.svelte
index 0a3c5a2101..8463f99825 100644
--- a/src/lib/layout/header.svelte
+++ b/src/lib/layout/header.svelte
@@ -124,8 +124,7 @@
         {/if}
         <DropList show={$feedback.show} scrollable on:blur={toggleFeedback}>
             <button class="button is-small is-text" on:click={toggleFeedback}>
-                <!-- TODO: invert this before release -->
-                {#if !$feedback.notification}
+                {#if $feedback.notification}
                     <span
                         class="notification u-position-absolute u-inset-block-start-8 u-inset-inline-end-8"
                     ></span>