change implementation

david-yz-liu · Sep 17, 2024 · c824ff1 · c824ff1
1 parent ce08f23
commit c824ff1
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/demo/src/App.tsx b/demo/src/App.tsx
@@ -71,10 +71,7 @@ export default function App() {
                             setSvgResult={setSvgResult}
                         />
                     </ErrorBoundary>
-                    <DownloadSVGButton
-                        svgResult={svgResult}
-                        disabled={!jsonResult}
-                    />
+                    <DownloadSVGButton svgResult={svgResult} />
                 </Box>
             </Stack>
         </main>

diff --git a/demo/src/DownloadSVGButton.tsx b/demo/src/DownloadSVGButton.tsx
@@ -3,7 +3,6 @@ import { Button } from "@mui/material";
 
 type DownloadSVGButtonPropTypes = {
     svgResult: string;
-    disabled: boolean;
 };
 export default function DownloadSVGButton(props: DownloadSVGButtonPropTypes) {
     const file = new global.Blob([props.svgResult], { type: "image/svg+xml" });
@@ -13,7 +12,7 @@ export default function DownloadSVGButton(props: DownloadSVGButtonPropTypes) {
             color="primary"
             variant="text"
             data-testid="download-svg-btn"
-            disabled={props.disabled}
+            disabled={!props.svgResult}
             href={URL.createObjectURL(file)}
             target="_blank"
             rel="noreferrer"

diff --git a/demo/src/SvgDisplay.tsx b/demo/src/SvgDisplay.tsx
@@ -27,6 +27,8 @@ export default function SvgDisplay(props: SvgDisplayPropTypes) {
             props.setSvgResult(m.serializeSVG());
             m.clear(canvasRef.current);
             m.render(canvasRef.current);
+        } else {
+            props.setSvgResult(null);
         }
     }, [props.jsonResult]);