From 53d87b7b96c6b218fc1aa45b31e148b1da452222 Mon Sep 17 00:00:00 2001
From: "Laurie T. Malau" <laurie@gitpod.io>
Date: Wed, 18 May 2022 14:01:37 +0000
Subject: [PATCH] ui fixes

---
 components/dashboard/src/Menu.tsx             |  2 +-
 .../src/feedback-form/FeedbackComponent.tsx   | 77 +++++++------------
 2 files changed, 28 insertions(+), 51 deletions(-)

diff --git a/components/dashboard/src/Menu.tsx b/components/dashboard/src/Menu.tsx
index 5c330b0b4f9928..9b5afb0bdb2a1b 100644
--- a/components/dashboard/src/Menu.tsx
+++ b/components/dashboard/src/Menu.tsx
@@ -395,11 +395,11 @@ export default function Menu() {
                                     {
                                         title: "Settings",
                                         link: "/settings",
-                                        separator: true,
                                     },
                                     {
                                         title: "Help",
                                         link: "https://www.gitpod.io/support",
+                                        separator: true,
                                     },
                                     {
                                         title: "Logout",
diff --git a/components/dashboard/src/feedback-form/FeedbackComponent.tsx b/components/dashboard/src/feedback-form/FeedbackComponent.tsx
index 8fe2d9a917bf15..a59d8307089cc0 100644
--- a/components/dashboard/src/feedback-form/FeedbackComponent.tsx
+++ b/components/dashboard/src/feedback-form/FeedbackComponent.tsx
@@ -105,55 +105,30 @@ function FeedbackComponent(props: { onClose: () => void; onSubmit: () => void })
         <>
             <h3 className="mb-4">Send Feedback</h3>
             {selectedEmoji ? (
-                <div className="flex flex-col -mx-6 px-6 py-4 border-t border-b border-gray-200 dark:border-gray-800">
-                    <div className="relative">
-                        <div className="absolute flex bottom-5 right-5 gap-3">{emojiGroup(24)}</div>
-                        <textarea
-                            style={{ height: "160px", borderRadius: "6px" }}
-                            autoFocus
-                            className="w-full"
-                            name="name"
-                            value={text}
-                            placeholder="Your feedback..."
-                            onChange={(e) => setText(e.target.value)}
-                        />
-                    </div>
-                    <div>
-                        <p className="pt-4">
-                            Alternatively, consider opening{" "}
-                            <a
-                                className="gp-link"
-                                href="https://github.com/gitpod-io/gitpod/issues/new?assignees=&labels=type%3A+bug&template=bug_report.yml"
-                                target="_blank"
-                                rel="noreferrer"
-                            >
-                                {" "}
-                                a bug report{" "}
-                            </a>
-                            or
-                            <a
-                                className="gp-link"
-                                href="https://github.com/gitpod-io/gitpod/issues/new?assignees=&labels=&template=feature_request.md&title="
-                                target="_blank"
-                                rel="noreferrer"
-                            >
+                <>
+                    <div className="flex flex-col -mx-6 px-6 py-4 border-t border-b border-gray-200 dark:border-gray-800 bg-gray-100 dark:bg-gray-900">
+                        <div className="relative">
+                            <div className="absolute flex bottom-5 right-5">{emojiGroup(24)}</div>
+                            <textarea
+                                style={{ height: "160px", borderRadius: "6px" }}
+                                autoFocus
+                                className="w-full resize-none text-gray-500 dark:text-gray-400"
+                                name="name"
+                                value={text}
+                                placeholder="Have more feedback?"
+                                onChange={(e) => setText(e.target.value)}
+                            />
+                        </div>
+                        <div>
+                            <p className="text-gray-500">
                                 {" "}
-                                a feature request{" "}
-                            </a>{" "}
-                            in our issue tracker.
-                        </p>
-                        <p className="pt-4">
-                            {" "}
-                            By submitting this form I acknowledge that I have read and understood Gitpod’s{" "}
-                            <a
-                                className="gp-link hover:text-gray-600"
-                                target="gitpod-privacy"
-                                href="https://www.gitpod.io/privacy/"
-                            >
-                                privacy policy
-                            </a>
-                            .
-                        </p>
+                                By submitting this form you acknowledge that you have read and understood Gitpod’s{" "}
+                                <a className="gp-link" target="gitpod-privacy" href="https://www.gitpod.io/privacy/">
+                                    privacy policy
+                                </a>
+                                .
+                            </p>
+                        </div>
                     </div>
                     <div className="flex justify-end mt-6">
                         <button className="secondary" onClick={props.onClose}>
@@ -163,10 +138,12 @@ function FeedbackComponent(props: { onClose: () => void; onSubmit: () => void })
                             Send Feedback
                         </button>
                     </div>
-                </div>
+                </>
             ) : (
                 <div className="flex flex-col -mx-6 px-6 py-4 border-t border-gray-200 dark:border-gray-800">
-                    <h4 className="text-center text-xl mb-4">We'd love to know what you think!</h4>
+                    <p className="text-center text-lg mb-8 text-gray-500 dark:text-gray-400">
+                        We'd love to know what you think!
+                    </p>
 
                     <div className="flex items-center justify-center w-full space-x-3">{emojiGroup(50)}</div>
                 </div>