From 4c2c0979bc259126d256c90bf5172e42a01b69e5 Mon Sep 17 00:00:00 2001
From: Shelley Vohr <shelley.vohr@gmail.com>
Date: Thu, 14 Dec 2023 17:10:15 +0100
Subject: [PATCH 1/2] fix: version filter should be a <select>

---
 src/static/css/releases.css   | 10 ++++++++++
 src/views/releases.handlebars | 23 +++++++++++++++--------
 2 files changed, 25 insertions(+), 8 deletions(-)

diff --git a/src/static/css/releases.css b/src/static/css/releases.css
index ee094c1..bc60b3f 100644
--- a/src/static/css/releases.css
+++ b/src/static/css/releases.css
@@ -68,6 +68,16 @@
   box-shadow: var(--shadow-elevation-medium);
 }
 
+#version-select {
+  border: 1px solid whitesmoke;
+  padding: 5px;
+  border-radius: 5px;
+  margin: 10px;
+  font-size: 20px;
+  font-weight: bolder;
+  color: var(--electron-color-dark);
+}
+
 .releases__filter--selected {
   border: 1px solid whitesmoke;
   border-left: 6px solid var(--electron-color-light);
diff --git a/src/views/releases.handlebars b/src/views/releases.handlebars
index 75632c1..ca85b07 100644
--- a/src/views/releases.handlebars
+++ b/src/views/releases.handlebars
@@ -22,14 +22,14 @@
         </div>
       </a>
     </div>
-  </div>
-  <div class="release-filter">
     <strong>Filter by major release</strong>
-    <div>
-    <a href="?">All</a>
-      {{#each majors}}
-        <a href="?version={{this}}" {{#ifEquals this ../major}}class="selected-release-filter"{{/ifEquals}}>{{this}}.x</a>
-      {{/each}}
+    <div class="release-version-selector-container">
+      <select id="version-select" onchange="showSelectedVersion('{{this.channel}}')">
+        <option value="All">All</option>
+        {{#each majors}}
+        <option value="{{this}}" {{#ifEquals this ../major}}selected{{/ifEquals}}>{{this}}.x</option>
+        {{/each}}
+      </select>
     </div>
   </div>
   </div>
@@ -41,6 +41,13 @@
       </article>
     {{/each}}
   </div>
-
 </div>
+
+<script>
+  function showSelectedVersion(channel) {
+    const { value: version } = document.getElementById("version-select");
+    window.location.href = version === 'All' ?`/releases/${channel}` : `/releases/${channel}?version=${version}`;
+  }
+</script>
+
 {{{paginate this.pages this.page this.prev this.next this.first this.last}}}

From eec1129fa064d17856dffe2082a36a94fd91b9f7 Mon Sep 17 00:00:00 2001
From: Shelley Vohr <shelley.vohr@gmail.com>
Date: Thu, 14 Dec 2023 19:16:27 +0100
Subject: [PATCH 2/2] fix: change window.location.search instead

---
 src/views/releases.handlebars | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/views/releases.handlebars b/src/views/releases.handlebars
index ca85b07..dc4160d 100644
--- a/src/views/releases.handlebars
+++ b/src/views/releases.handlebars
@@ -24,7 +24,7 @@
     </div>
     <strong>Filter by major release</strong>
     <div class="release-version-selector-container">
-      <select id="version-select" onchange="showSelectedVersion('{{this.channel}}')">
+      <select id="version-select" onchange="showSelectedVersion()">
         <option value="All">All</option>
         {{#each majors}}
         <option value="{{this}}" {{#ifEquals this ../major}}selected{{/ifEquals}}>{{this}}.x</option>
@@ -44,9 +44,9 @@
 </div>
 
 <script>
-  function showSelectedVersion(channel) {
+  function showSelectedVersion() {
     const { value: version } = document.getElementById("version-select");
-    window.location.href = version === 'All' ?`/releases/${channel}` : `/releases/${channel}?version=${version}`;
+    window.location.search = version === 'All' ? '' : `version=${version}`;
   }
 </script>