Simplify & fix bugs with select all functionality

themes/alps/assets/helpers.js

@@ -1,24 +1,10 @@
-(function() {
-	var sheet = document.styleSheets[0];
-	var addCSSRule = function(selector, rules, index) {
-		if ("insertRule" in sheet) {
-			sheet.insertRule(selector + "{" + rules + "}", index);
-			return;
+const check_all = document.getElementById("action-checkbox-all");
+if (check_all) {
+	check_all.style.display = "inherit";
+	check_all.addEventListener("click", ev => {
+		const inputs = document.querySelectorAll(".message-list-checkbox input");
+		for (let i = 0; i < inputs.length; i++) {
+			inputs[i].checked = ev.target.checked;
 		}
-		if ("addRule" in sheet) {
-			sheet.addRule(selector, rules, index);
-		}
-	};
-
-	var checkboxAll = document.getElementById("action-checkbox-all");
-	if (checkboxAll) {
-		addCSSRule(".message-list-checkbox", "display: table-cell !important;");
-		checkboxAll.addEventListener("click", function(ev) {
-			var allChecked = this.checked;
-			var inputs = document.querySelectorAll("tr .message-list-checkbox input");
-			Array.prototype.slice.apply().forEach(function(cb) {
-				cb.checked = allChecked;
-			});
-		});
-	}
-})();
+	});
+}

themes/alps/mailbox.html

@@ -2,25 +2,6 @@
 {{template "nav.html" .}}
 {{template "util.html" .}}
 
-{{ define "mbox-link" }}
-{{ if not (.HasAttr "\\Noselect") }}
-<a href="{{.URL}}" {{ if .Active }}class="active"{{ end }}>
-  {{- if eq .Name "INBOX" -}}
-    Inbox
-  {{- else -}}
-    {{ .Name }}
-  {{- end -}}
-  {{- if .HasAttr "\\HasChildren" }}/{{ end }}
-
-  {{ if and (ne .Unseen -1) (ne .Unseen 0) }}({{ .Unseen }}){{ end }}
-</a>
-{{ else }}
-<span class="noselect">
-  {{.Name}}{{- if .HasAttr "\\HasChildren" }}/{{ end }}
-</span>
-{{ end }}
-{{ end }}
-
 <div class="page-wrap">
   {{ template "aside" . }}
   <div class="container">

themes/alps/messages-header.html

@@ -1,5 +1,5 @@
 <div class="message-list-checkbox">
-  <input type="checkbox" id="action-checkbox-all"/>
+  <input type="checkbox" id="action-checkbox-all" style="display: none"/>
 </div>
 <div class="actions-wrap">
   <div class="actions-message">