From 0aa50d50a6ba5ce528bde8c87a957f11f0783607 Mon Sep 17 00:00:00 2001
From: Nayam Amarshe <25067102+NayamAmarshe@users.noreply.github.com>
Date: Mon, 19 Sep 2022 16:44:40 +0530
Subject: [PATCH] Polished UI

---
 main/index.js                         |  2 +-
 renderer/components/LeftPaneSteps.jsx | 82 ++++++++++++++-------------
 renderer/styles/globals.css           |  6 ++
 3 files changed, 50 insertions(+), 40 deletions(-)

diff --git a/main/index.js b/main/index.js
index 47534a4..821a2b9 100644
--- a/main/index.js
+++ b/main/index.js
@@ -35,7 +35,7 @@ app.on("ready", async () => {
   mainWindow = new BrowserWindow({
     icon: join(__dirname, "build", "icon.png"),
     width: 1100,
-    height: 700,
+    height: 740,
     minHeight: 500,
     minWidth: 500,
     show: false,
diff --git a/renderer/components/LeftPaneSteps.jsx b/renderer/components/LeftPaneSteps.jsx
index c053218..1fa4975 100644
--- a/renderer/components/LeftPaneSteps.jsx
+++ b/renderer/components/LeftPaneSteps.jsx
@@ -11,7 +11,7 @@ function LeftPaneSteps(props) {
       {/* BATCH OPTION */}
       <div className="flex flex-row items-end">
         <p
-          className="mr-1 inline-block cursor-help text-sm text-neutral-400"
+          className="mr-1 inline-block cursor-help text-sm text-white/70"
           data-tip="This will let you upscale all files in a folder at once"
         >
           Batch Upscale:
@@ -32,7 +32,7 @@ function LeftPaneSteps(props) {
 
       {/* STEP 1 */}
       <div data-tip={props.imagePath}>
-        <p className="mb-2 font-medium text-white/70">Step 1</p>
+        <p className="step-heading">Step 1</p>
         <button
           className="bg-gradient-red rounded-lg p-3 font-medium text-white/90 transition-colors"
           onClick={
@@ -47,37 +47,9 @@ function LeftPaneSteps(props) {
 
       {/* STEP 2 */}
       <div className="animate-step-in">
-        <p className="font-medium text-neutral-100">Step 2</p>
-        <p className="mb-2 text-sm text-neutral-400">Select Upscaling Type</p>
-        {props.model !== "models-DF2K" && !props.batchMode && (
-          <div className="mb-2 flex items-center gap-1">
-            <input
-              type="checkbox"
-              className="checked:bg-gradient-white h-4 w-4 cursor-pointer appearance-none rounded-full bg-neutral-500 transition duration-200 focus:outline-none focus-visible:border focus-visible:border-green-400"
-              checked={props.doubleUpscayl}
-              onChange={(e) => {
-                if (e.target.checked) {
-                  props.setDoubleUpscayl(true);
-                } else {
-                  props.setDoubleUpscayl(false);
-                }
-              }}
-            />
-            <p
-              data-tip="Enable this option to get an 8x upscayl"
-              className={`inline-block cursor-help rounded-full text-sm font-medium ${
-                props.doubleUpscayl
-                  ? "bg-gradient-white px-2 text-black/90"
-                  : "text-neutral-500"
-              }`}
-              onClick={(e) => {
-                props.setDoubleUpscayl(!props.doubleUpscayl);
-              }}
-            >
-              Double Upscayl
-            </p>
-          </div>
-        )}
+        <p className="step-heading">Step 2</p>
+        <p className="mb-2 text-sm text-white/60">Select Upscaling Type</p>
+
         <select
           name="select-model"
           onDrop={(e) => props.handleDrop(e)}
@@ -88,14 +60,46 @@ function LeftPaneSteps(props) {
           <option value="realesrgan-x4plus-anime">Digital Art</option>
           <option value="models-DF2K">Sharpen Image</option>
         </select>
+        {props.model !== "models-DF2K" && !props.batchMode && (
+          <div className="mt-2 flex items-center gap-1">
+            <input
+              type="checkbox"
+              className="checked:bg-gradient-white h-4 w-4 cursor-pointer appearance-none rounded-full bg-white/30 transition duration-200 focus:outline-none focus-visible:border focus-visible:border-green-400"
+              checked={props.doubleUpscayl}
+              onChange={(e) => {
+                if (e.target.checked) {
+                  props.setDoubleUpscayl(true);
+                } else {
+                  props.setDoubleUpscayl(false);
+                }
+              }}
+            />
+            <p
+              className={`inline-block cursor-pointer select-none rounded-full text-sm font-medium ${
+                props.doubleUpscayl
+                  ? "bg-gradient-white px-2 text-black/90"
+                  : "text-white/50"
+              }`}
+              onClick={(e) => {
+                props.setDoubleUpscayl(!props.doubleUpscayl);
+              }}
+            >
+              Double Upscayl
+            </p>
+            <span
+              className="cursor-help rounded-full bg-white/20 px-3 text-center font-bold text-white/40"
+              data-tip="Enable this option to get an 8x upscayl. Note that this may not always work properly with all images, for example, images with really large resolutions."
+            >
+              i
+            </span>
+          </div>
+        )}
       </div>
 
       {/* STEP 3 */}
       <div className="animate-step-in" data-tip={props.outputPath}>
-        <p className="font-medium text-neutral-100">Step 3</p>
-        <p className="mb-2 text-sm text-neutral-400">
-          Defaults to Image's path
-        </p>
+        <p className="step-heading">Step 3</p>
+        <p className="mb-2 text-sm text-white/60">Defaults to Image's path</p>
         <button
           className="bg-gradient mt-1 rounded-lg p-3 font-medium text-black/90 transition-colors hover:bg-teal-300"
           onClick={props.outputHandler}
@@ -106,7 +110,7 @@ function LeftPaneSteps(props) {
 
       {/* STEP 4 */}
       <div className="animate-step-in">
-        <p className="mb-2 font-medium text-neutral-100">Step 4</p>
+        <p className="step-heading">Step 4</p>
         <button
           className="bg-gradient-upscayl rounded-lg p-3 font-medium text-white/90 transition-colors"
           onClick={props.upscaylHandler}
@@ -117,7 +121,7 @@ function LeftPaneSteps(props) {
       </div>
 
       <ReactTooltip
-        className="max-w-72 break-words bg-neutral-900 text-neutral-50"
+        className="max-w-md break-words bg-neutral-900 text-neutral-50"
         place="top"
       />
     </div>
diff --git a/renderer/styles/globals.css b/renderer/styles/globals.css
index 37f2e42..3395cba 100644
--- a/renderer/styles/globals.css
+++ b/renderer/styles/globals.css
@@ -3,6 +3,9 @@
 @tailwind utilities;
 
 @layer base {
+  * {
+    @apply select-none;
+  }
   ::-webkit-scrollbar {
     @apply w-3;
   }
@@ -42,6 +45,9 @@
   .bg-gradient-blue {
     @apply bg-gradient-to-br from-blue-500 to-sky-400 hover:from-sky-400 hover:to-blue-500;
   }
+  .step-heading {
+    @apply mb-2 font-medium text-white/80;
+  }
 }
 
 .animate-step-in {