From 5b3421bf6fc4661b46612300bf44f30495b9c09e Mon Sep 17 00:00:00 2001
From: drafish <xwlyy1991@163.com>
Date: Tue, 22 Oct 2024 13:40:30 +0800
Subject: [PATCH] add delete logo button in quick-dapp

---
 .../src/components/ImageUpload/index.tsx      | 53 +++++++++++++++----
 .../src/components/DappTop/index.tsx          |  2 +-
 .../src/app/tabs/locales/en/quickDapp.json    |  4 +-
 3 files changed, 46 insertions(+), 13 deletions(-)

diff --git a/apps/quick-dapp/src/components/ImageUpload/index.tsx b/apps/quick-dapp/src/components/ImageUpload/index.tsx
index 74deb54a74..f4a2a20f29 100644
--- a/apps/quick-dapp/src/components/ImageUpload/index.tsx
+++ b/apps/quick-dapp/src/components/ImageUpload/index.tsx
@@ -31,18 +31,49 @@ const ImageUpload = () => {
   return (
     <div className="col-3 pr-0">
       <input data-id="uploadLogo" className="d-none" type="file" accept="image/*" onChange={handleImageChange} id="upload-button" />
-      <CustomTooltip
-        placement="right"
-        tooltipText={intl.formatMessage({ id: 'quickDapp.uploadLogoTooltip' })}
-      >
-        <label htmlFor="upload-button" className="cursor_pointer d-flex justify-content-center align-items-center position-relative" style={{ height: 170 }}>
-          {logo ? (
-            <img src={preview} alt="preview" style={{ width: 120, height: 120 }} />
-          ) : (
+      <label htmlFor="upload-button" className="cursor_pointer d-flex justify-content-center align-items-center position-relative" style={{ height: 170 }}>
+        {logo ? (
+          <>
+            <CustomTooltip
+              placement="right"
+              tooltipText={intl.formatMessage({ id: 'quickDapp.updateLogoTooltip' })}
+            >
+              <img src={preview} alt="preview" style={{ width: 120, height: 120 }} />
+            </CustomTooltip>
+            <CustomTooltip
+              placement="right"
+              tooltipText={intl.formatMessage({ id: 'quickDapp.deleteLogoTooltip' })}
+            >
+              <button
+                className="d-flex justify-content-center align-items-center position-absolute border-0 rounded-circle"
+                style={{
+                  top: 5,
+                  right: 5,
+                  width: 20,
+                  height: 20,
+                  backgroundColor: 'var(--gray-dark)'
+                }}
+                onClick={(event) => {
+                  event.preventDefault()
+                  dispatch({ type: 'SET_INSTANCE', payload: { logo: '' } })
+                }}
+              >
+                <i className="fas fa-times" style={{
+                  color: 'var(--text-bg-mark)',
+                  fontSize: 'large'
+                }}></i>
+              </button>
+            </CustomTooltip>
+          </>
+        ) : (
+          <CustomTooltip
+            placement="right"
+            tooltipText={intl.formatMessage({ id: 'quickDapp.addLogoTooltip' })}
+          >
             <i className="fas fa-upload" style={{ fontSize: 120 }}></i>
-          )}
-        </label>
-      </CustomTooltip>
+          </CustomTooltip>
+        )}
+      </label>
     </div>
   )
 }
diff --git a/apps/remix-dapp/src/components/DappTop/index.tsx b/apps/remix-dapp/src/components/DappTop/index.tsx
index 4fbdd61f2e..1838e65361 100644
--- a/apps/remix-dapp/src/components/DappTop/index.tsx
+++ b/apps/remix-dapp/src/components/DappTop/index.tsx
@@ -30,7 +30,7 @@ const DappTop: React.FC = () => {
   const shareTitle = encodeURIComponent('Hello everyone, this is my dapp!');
 
   return (
-    <div className={`${instance.logo ? 'col-10' : 'col-12'} p-3 bg-light w-auto d-flex justify-content-between`}>
+    <div className={`${instance.showLogo ? 'col-10' : 'col-12'} p-3 bg-light w-auto d-flex justify-content-between`}>
       <div>
         {title && <h1 data-id="dappTitle">{title}</h1>}
         {details && <span data-id="dappInstructions">{details}</span>}
diff --git a/apps/remix-ide/src/app/tabs/locales/en/quickDapp.json b/apps/remix-ide/src/app/tabs/locales/en/quickDapp.json
index 1a7164f261..78e32b6b34 100644
--- a/apps/remix-ide/src/app/tabs/locales/en/quickDapp.json
+++ b/apps/remix-ide/src/app/tabs/locales/en/quickDapp.json
@@ -32,7 +32,9 @@
   "quickDapp.text4": "Deployed successfully!",
   "quickDapp.text5": "Click the link below to view your dapp",
   "quickDapp.text6": "Teardown successfully!",
-  "quickDapp.uploadLogoTooltip": "Click here to change logo",
+  "quickDapp.addLogoTooltip": "Click here to add a logo",
+  "quickDapp.updateLogoTooltip": "Click here to update the logo",
+  "quickDapp.deleteLogoTooltip": "Click here to delete the logo",
   "quickDapp.dappTitle": "Dapp Title",
   "quickDapp.dappInstructions": "Dapp Instructions",
   "quickDapp.functionTitle": "Title of function",