From 144c816be3fb5a97e4b871668a4730c7b70bff71 Mon Sep 17 00:00:00 2001
From: winkidney <winkidney@gmail.com>
Date: Thu, 5 Mar 2020 12:19:31 +0800
Subject: [PATCH] Fix: Should not use 'form' element to avoid request-cancel

Also fix default value for privacy option of board and pin.

Ref: https://stackoverflow.com/questions/12009423/what-does-status-canceled-for-a-resource-mean-in-chrome-developer-tools
---
 pinry-spa/src/components/BoardEdit.vue               | 6 ++++--
 pinry-spa/src/components/LoginForm.vue               | 4 ++--
 pinry-spa/src/components/SignUpForm.vue              | 4 ++--
 pinry-spa/src/components/pin_edit/Add2Board.vue      | 4 ++--
 pinry-spa/src/components/pin_edit/PinCreateModal.vue | 7 ++++---
 5 files changed, 14 insertions(+), 11 deletions(-)

diff --git a/pinry-spa/src/components/BoardEdit.vue b/pinry-spa/src/components/BoardEdit.vue
index ecb34b3..35f06e7 100644
--- a/pinry-spa/src/components/BoardEdit.vue
+++ b/pinry-spa/src/components/BoardEdit.vue
@@ -1,6 +1,6 @@
 <template>
   <div class="board-modal">
-    <form action="">
+    <div>
       <div class="modal-card" style="width: auto">
         <header class="modal-card-head">
           <p class="modal-card-title">{{ UIMeta.title }}</p>
@@ -61,7 +61,7 @@
           </button>
         </footer>
       </div>
-    </form>
+    </div>
   </div>
 </template>
 
@@ -101,6 +101,8 @@ export default {
     if (this.isEdit) {
       this.UIMeta.title = 'Board Edit';
       this.editModel.assignToForm(this.board);
+    } else {
+      this.createModel.form.private.value = false;
     }
   },
   methods: {
diff --git a/pinry-spa/src/components/LoginForm.vue b/pinry-spa/src/components/LoginForm.vue
index 68435dd..d689e0b 100644
--- a/pinry-spa/src/components/LoginForm.vue
+++ b/pinry-spa/src/components/LoginForm.vue
@@ -1,6 +1,6 @@
 <template>
   <div class="login-modal">
-    <form action="">
+    <div>
       <div class="modal-card" style="width: auto">
         <header class="modal-card-head">
           <p class="modal-card-title">Login</p>
@@ -38,7 +38,7 @@
             class="button is-primary">Login</button>
         </footer>
       </div>
-    </form>
+    </div>
   </div>
 </template>
 
diff --git a/pinry-spa/src/components/SignUpForm.vue b/pinry-spa/src/components/SignUpForm.vue
index 1c24e2b..ebd1d87 100644
--- a/pinry-spa/src/components/SignUpForm.vue
+++ b/pinry-spa/src/components/SignUpForm.vue
@@ -1,6 +1,6 @@
 <template>
   <div class="signup-modal">
-    <form action="">
+    <div>
       <div class="modal-card" style="width: auto">
         <header class="modal-card-head">
           <p class="modal-card-title">Sign Up</p>
@@ -59,7 +59,7 @@
             class="button is-primary">Register</button>
         </footer>
       </div>
-    </form>
+    </div>
   </div>
 </template>
 
diff --git a/pinry-spa/src/components/pin_edit/Add2Board.vue b/pinry-spa/src/components/pin_edit/Add2Board.vue
index a91e120..66eeaf3 100644
--- a/pinry-spa/src/components/pin_edit/Add2Board.vue
+++ b/pinry-spa/src/components/pin_edit/Add2Board.vue
@@ -1,6 +1,6 @@
 <template>
   <div class="add2board-modal">
-    <form action="">
+    <div>
       <div class="modal-card" style="width: auto">
         <header class="modal-card-head">
           <p class="modal-card-title">Add Pin to Board</p>
@@ -28,7 +28,7 @@
           </button>
         </footer>
       </div>
-    </form>
+    </div>
   </div>
 </template>
 
diff --git a/pinry-spa/src/components/pin_edit/PinCreateModal.vue b/pinry-spa/src/components/pin_edit/PinCreateModal.vue
index 0abfaa9..2115ce5 100644
--- a/pinry-spa/src/components/pin_edit/PinCreateModal.vue
+++ b/pinry-spa/src/components/pin_edit/PinCreateModal.vue
@@ -1,6 +1,6 @@
 <template>
   <div class="pin-create-modal">
-    <form action="">
+    <div>
       <div class="modal-card" style="width: auto">
         <header class="modal-card-head">
           <p class="modal-card-title">{{ editorMeta.title }}</p>
@@ -97,7 +97,7 @@
           </button>
         </footer>
       </div>
-    </form>
+    </div>
   </div>
 </template>
 
@@ -168,12 +168,13 @@ export default {
       this.pinModel.form.description.value = this.existedPin.description;
       this.pinModel.form.tags.value = this.existedPin.tags;
       this.pinModel.form.private.value = this.existedPin.private;
+    } else {
+      this.pinModel.form.private.value = false;
     }
     if (this.fromUrl) {
       this.pinModel.form.url.value = this.fromUrl.url;
       this.pinModel.form.referer.value = this.fromUrl.referer;
       this.pinModel.form.description.value = this.fromUrl.description;
-      this.pinModel.form.private.value = false;
     }
   },
   methods: {