Browse Source

fix: check `href` null

YuKun Liu 3 years ago
parent
commit
327f901
1 changed files with 7 additions and 5 deletions
  1. 7 5
      packages/desktop/src/index.js

+ 7 - 5
packages/desktop/src/index.js

@@ -364,12 +364,14 @@ class Interpreter {
         if (event.type == "click") {
         if (event.type == "click") {
           event.preventDefault();
           event.preventDefault();
           if (should_prevent_default !== `onclick`) {
           if (should_prevent_default !== `onclick`) {
-            console.log(event.target.getAttribute("href"));
             if(element.tagName == "A") {
             if(element.tagName == "A") {
-              rpc.call("browser_open", {
-                mounted_dom_id: parseInt(real_id),
-                href: event.target.getAttribute("href")
-              })
+              const href = event.target.getAttribute("href")
+              if (href !== "" || href !== null) {
+                rpc.call("browser_open", {
+                  mounted_dom_id: parseInt(real_id),
+                  href: event.target.getAttribute("href")
+                });
+              }
             }
             }
           }
           }
         }
         }