From a4f9e4b5629f00036a02621721649d3749bda249 Mon Sep 17 00:00:00 2001
From: WofWca <wofwca@protonmail.com>
Date: Mon, 5 Jan 2026 16:25:00 +0400
Subject: [PATCH] fix: turn app `<button>`s into `<a>`s

This allows Shift / Ctrl + Clicking them to open in new tab
and is overall more sensible with our `hashchange` approach.

A side effect of this is that the app list items
no longer change brightness on hover,
which could be considered a positive side effect by some.

Apparently the initial implementation of showing apps in a dialog
did not have the `hashchange` handler yet, so that was probably
the reason to use `<button>` instead of `<a>`:
https://github.com/webxdc/website/pull/99/changes/58d8e444a0423563e9316b6d97b18b9a9b5fd870.
The `hashchange` listener was added a few commits later:
https://github.com/webxdc/website/pull/99/changes/7b294dc62b8a045f7c13a600a5a9ab290ce1e074.
---
 website/apps/main.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/website/apps/main.js b/website/apps/main.js
index c19609c..817ec05 100644
--- a/website/apps/main.js
+++ b/website/apps/main.js
@@ -27,9 +27,9 @@ const App = ({ app, toggleModal }) => {
   const subtitle = app.description.split('\n').shift();
 
   return html`
-    <button
+    <a
       class="app"
-      onClick=${() => toggleModal(app.app_id)}
+      href="#${app.app_id}"
       key=${app.app_id}>
       <img src=${xdcget_export + "/" + app.icon_relname} loading="lazy" alt="Icon for ${app.name} app" />
       <div class="props ellipse">