refactor(defaultTheme): refine enableNonRefreshDelete()

Author: marjune163

src/tpl/defaultTheme/frontend/index.html

@@ -78,12 +78,12 @@ <h3>{{.Trans.UploadLabel}}</h3>
 
 {{if .CanDelete}}
 <script type="text/javascript">
-	function confirmDelete(form) {
+	function confirmDelete(e) {
+		var form = e.target;
 		var name = form.name.value;
 		var proceed = confirm('{{.Trans.DeleteConfirm}}\n' + name);
 		if (!proceed) {
-			var e = event || window.event;
-			if (e && !('defaultPrevented' in e)) e.defaultPrevented = true;
+			e.preventDefault();
 		}
 		return proceed;
 	}
@@ -124,7 +124,7 @@ <h3>{{.Trans.UploadLabel}}</h3>
 			<span class="field size">{{.DisplaySize}}</span>
 			<span class="field time">{{.DisplayTime}}</span>
 		</a>
-		{{if and (not $isSimple) .DeleteUrl}}<form class="delete" method="post" action="{{$SubItemPrefix}}?delete" onsubmit="return confirmDelete(this)"><input type="hidden" name="name" value="{{.DeleteUrl}}"/><input type="hidden" name="contextquerystring" value="{{$contextQueryString}}"/><button type="submit">x</button></form>{{end}}
+		{{if and (not $isSimple) .DeleteUrl}}<form class="delete" method="post" action="{{$SubItemPrefix}}?delete" onsubmit="return confirmDelete(event)"><input type="hidden" name="name" value="{{.DeleteUrl}}"/><input type="hidden" name="contextquerystring" value="{{$contextQueryString}}"/><button type="submit">x</button></form>{{end}}
 	</li>
 	{{end}}
 </ul>

src/tpl/defaultTheme/frontend/index.js

@@ -980,32 +980,23 @@
 		if (!itemList.classList.contains('has-deletable')) return;
 
 		itemList.addEventListener('submit', function (e) {
-			if (e.defaultPrevented) {
-				return;
-			}
+			if (e.defaultPrevented) return;
 
 			var form = e.target;
 
 			function onLoad() {
 				var status = this.status;
-				if (status >= 200 && status <= 299) {
-					var elItem = form;
-					while (elItem && elItem.nodeName !== 'LI') {
-						elItem = elItem.parentElement;
-					}
-					if (!elItem) {
-						return;
-					}
-					var elItemParent = elItem.parentNode;
-					elItemParent && elItemParent.removeChild(elItem);
+				if (status >= 200 && status < 300) {
+					var elItem = form.closest('li');
+					elItem.remove();
 				} else {
 					logError('delete failed: ' + status + ' ' + this.statusText);
 				}
 			}
 
 			var params = '';
 			var els = Array.prototype.slice.call(form.elements);
-			for (var i = 0, len = els.length; i < len; i++) {
+			for (var i = 0; i < els.length; i++) {
 				if (!els[i].name) {
 					continue
 				}
@@ -1023,7 +1014,7 @@
 			xhr.send(params);
 			e.preventDefault();
 			return false;
-		}, false);
+		});
 	}
 
 	enableFilter();