From 61d8a29b208d3067edce41b812d7481f54e31126 Mon Sep 17 00:00:00 2001
From: Ritvik Nag <nagrim7@gmail.com>
Date: Sun, 11 Feb 2024 13:37:33 -0500
Subject: [PATCH] fix: don't copy to clipboard if textbox is empty

---
 src/components/medium/medium.js       | 40 +++++++++++++--------------
 src/components/textfield/textfield.js |  6 ++--
 src/state.js                          |  2 +-
 3 files changed, 24 insertions(+), 24 deletions(-)

diff --git a/src/components/medium/medium.js b/src/components/medium/medium.js
index ef59087..0b25a87 100644
--- a/src/components/medium/medium.js
+++ b/src/components/medium/medium.js
@@ -3,31 +3,29 @@ import './medium.css';
 import { connect } from 'react-redux';
 
 const Medium = props => {
-    const { content, copySuccess } = props;
+    const { content: _, copySuccess } = props;
 
-    let text;
-
-    if (content) {
-        text = (
-            <div>
-                {(copySuccess && (
-                    <h3 style={{ color: 'green' }}>
-                        Copied to Clipboard!
+    const copyStatusMessage = (
+        <div>
+            {(copySuccess && (
+                <h3 style={{ color: 'green' }}>
+                    Copied to Clipboard!
+                </h3>
+            )) ||
+                (copySuccess === undefined && (
+                    <h3 style={{ color: 'gray' }}>
+                        Waiting to copy rich text...
                     </h3>
                 )) || <h3 style={{ color: 'red' }}>Error!</h3>}
-            </div>
-        );
-    } else {
-        text = (
-            <div>
-                <h3 style={{ color: 'gray' }}>
-                    Waiting to copy rich text...
-                </h3>
-            </div>
-        );
-    }
+        </div>
+    );
 
-    return <div id="medium" className="medium" children={text}></div>;
+    return (
+        <div
+            id="medium"
+            className="medium"
+            children={copyStatusMessage}></div>
+    );
 };
 
 export default connect(state => ({
diff --git a/src/components/textfield/textfield.js b/src/components/textfield/textfield.js
index 699f864..6c2e0d5 100644
--- a/src/components/textfield/textfield.js
+++ b/src/components/textfield/textfield.js
@@ -13,8 +13,10 @@ const TextField = props => {
         let copySuccess;
 
         try {
-            await copyRichToClip(converter(contentVal));
-            copySuccess = true;
+            if (contentVal) {
+                await copyRichToClip(converter(contentVal));
+                copySuccess = true;
+            } else copySuccess = undefined;
         } catch (e) {
             console.log('Fail', e);
             copySuccess = false;
diff --git a/src/state.js b/src/state.js
index ac0c5d3..2635e8f 100644
--- a/src/state.js
+++ b/src/state.js
@@ -9,7 +9,7 @@ export function setContent(content, copySuccess) {
 export function reducer(
     state = {
         content: '',
-        copySuccess: false,
+        copySuccess: undefined,
     },
     action
 ) {