From 1614fea76a6fc1f4d70ddb14b95e3004c79365ef Mon Sep 17 00:00:00 2001
From: Kit La Touche <kit@transneptune.net>
Date: Mon, 4 Nov 2024 22:31:27 -0500
Subject: Set up framework for testing touch events

This includes a change to tools/run that exposes the dev server on my
local network. This change should not make it into the final form of
this branch.

This is so I can use actual for real touch events on my actual for real
phone, hooked up for remote debugging to my computer so I can see
console events etc.
---
 ui/routes/(app)/+layout.svelte | 35 +++++++++++++++++++++++++++++++++++
 1 file changed, 35 insertions(+)

(limited to 'ui')

diff --git a/ui/routes/(app)/+layout.svelte b/ui/routes/(app)/+layout.svelte
index ae3dc6a..acaea53 100644
--- a/ui/routes/(app)/+layout.svelte
+++ b/ui/routes/(app)/+layout.svelte
@@ -25,6 +25,40 @@
     messages.update((value) => value.setMessages(boot.messages));
   }
 
+  function setTouchEvents() {
+      document.addEventListener("touchstart", processTouchStart);
+      document.addEventListener("touchmove", processTouchMove);
+      document.addEventListener("touchcancel", processTouchCancel);
+      document.addEventListener("touchend", processTouchEnd);
+  }
+
+  function processTouchStart(ev) {
+      ev.preventDefault();
+      switch (ev.touches.length) {
+          case 1:
+              console.log("touch: single");
+              break;
+          case 2:
+              console.log("touch: double");
+              break;
+          case 3:
+              console.log("touch: triple");
+              break;
+          default:
+              console.log("touch: unsupported");
+              break;
+      }
+  }
+
+  function processTouchMove(ev) {
+  }
+
+  function processTouchCancel(ev) {
+  }
+
+  function processTouchEnd(ev) {
+  }
+
   onMount(async () => {
     let response = await boot();
     switch (response.status) {
@@ -45,6 +79,7 @@
         break;
     }
     loading = false;
+    setTouchEvents();
   });
 
   onDestroy(async () => {
-- 
cgit v1.2.3


From b13b5b617022a679d93128bbee6032ea1bb31147 Mon Sep 17 00:00:00 2001
From: Kit La Touche <kit@transneptune.net>
Date: Thu, 14 Nov 2024 16:59:24 -0500
Subject: Add in start of tests

---
 package-lock.json                                  | 362 +++++++++++++++++++++
 package.json                                       |   1 +
 .../lib/components/ChangePassword.svelte.test.js   |  28 ++
 .../components/CreateChannelForm.svelte.test.js    |   1 +
 ui/tests/lib/components/Invite.svelte.test.js      |   1 +
 ui/tests/lib/components/LogOut.svelte.test.js      |   1 +
 ui/tests/lib/components/Message.svelte.test.js     |   2 +
 .../lib/components/MessageInput.svelte.test.js     |   2 +
 vite.config.js                                     |   6 +
 9 files changed, 404 insertions(+)
 create mode 100644 ui/tests/lib/components/ChangePassword.svelte.test.js
 create mode 100644 ui/tests/lib/components/CreateChannelForm.svelte.test.js
 create mode 100644 ui/tests/lib/components/Invite.svelte.test.js
 create mode 100644 ui/tests/lib/components/LogOut.svelte.test.js
 create mode 100644 ui/tests/lib/components/Message.svelte.test.js
 create mode 100644 ui/tests/lib/components/MessageInput.svelte.test.js

(limited to 'ui')

diff --git a/package-lock.json b/package-lock.json
index ebc307d..335880f 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -26,6 +26,7 @@
 				"eslint-config-prettier": "^9.1.0",
 				"eslint-plugin-svelte": "^2.46.0",
 				"globals": "^15.11.0",
+				"jsdom": "^25.0.1",
 				"postcss": "^8.4.47",
 				"prettier": "^3.3.3",
 				"prettier-plugin-svelte": "^3.2.7",
@@ -911,6 +912,19 @@
 				"acorn": ">=8.9.0"
 			}
 		},
+		"node_modules/agent-base": {
+			"version": "7.1.1",
+			"resolved": "https://registry.npmjs.org/agent-base/-/agent-base-7.1.1.tgz",
+			"integrity": "sha512-H0TSyFNDMomMNJQBn8wFV5YC/2eJ+VXECwOadZJT554xP6cODZHPX3H9QMQECxvrgiSOP1pHjy1sMWQVYJOUOA==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"debug": "^4.3.4"
+			},
+			"engines": {
+				"node": ">= 14"
+			}
+		},
 		"node_modules/ajv": {
 			"version": "6.12.6",
 			"resolved": "https://registry.npmjs.org/ajv/-/ajv-6.12.6.tgz",
@@ -1374,6 +1388,33 @@
 				"node": ">=4"
 			}
 		},
+		"node_modules/cssstyle": {
+			"version": "4.1.0",
+			"resolved": "https://registry.npmjs.org/cssstyle/-/cssstyle-4.1.0.tgz",
+			"integrity": "sha512-h66W1URKpBS5YMI/V8PyXvTMFT8SupJ1IzoIV8IeBC/ji8WVmrO8dGlTi+2dh6whmdk6BiKJLD/ZBkhWbcg6nA==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"rrweb-cssom": "^0.7.1"
+			},
+			"engines": {
+				"node": ">=18"
+			}
+		},
+		"node_modules/data-urls": {
+			"version": "5.0.0",
+			"resolved": "https://registry.npmjs.org/data-urls/-/data-urls-5.0.0.tgz",
+			"integrity": "sha512-ZYP5VBHshaDAiVZxjbRVcFJpc+4xGgT0bK3vzy1HLN8jTO975HEbuYzZJcHoQEY5K1a0z8YayJkyVETa08eNTg==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"whatwg-mimetype": "^4.0.0",
+				"whatwg-url": "^14.0.0"
+			},
+			"engines": {
+				"node": ">=18"
+			}
+		},
 		"node_modules/debug": {
 			"version": "4.3.7",
 			"resolved": "https://registry.npmjs.org/debug/-/debug-4.3.7.tgz",
@@ -1392,6 +1433,13 @@
 				}
 			}
 		},
+		"node_modules/decimal.js": {
+			"version": "10.4.3",
+			"resolved": "https://registry.npmjs.org/decimal.js/-/decimal.js-10.4.3.tgz",
+			"integrity": "sha512-VBBaLc1MgL5XpzgIP7ny5Z6Nx3UrRkIViUkPUdtl9aya5amy3De1gsUUSB1g3+3sExYNjCAsAznmukyxCb1GRA==",
+			"dev": true,
+			"license": "MIT"
+		},
 		"node_modules/deep-eql": {
 			"version": "5.0.2",
 			"resolved": "https://registry.npmjs.org/deep-eql/-/deep-eql-5.0.2.tgz",
@@ -1476,6 +1524,19 @@
 			"dev": true,
 			"license": "MIT"
 		},
+		"node_modules/entities": {
+			"version": "4.5.0",
+			"resolved": "https://registry.npmjs.org/entities/-/entities-4.5.0.tgz",
+			"integrity": "sha512-V0hjH4dGPh9Ao5p0MoRY6BVqtwCjhz6vI5LT8AJ55H+4g9/4vbHx1I54fS0XuclLhDHArPQCiMjDxjaL8fPxhw==",
+			"dev": true,
+			"license": "BSD-2-Clause",
+			"engines": {
+				"node": ">=0.12"
+			},
+			"funding": {
+				"url": "https://github.com/fb55/entities?sponsor=1"
+			}
+		},
 		"node_modules/esbuild": {
 			"version": "0.21.5",
 			"resolved": "https://registry.npmjs.org/esbuild/-/esbuild-0.21.5.tgz",
@@ -2120,6 +2181,19 @@
 				"node": ">= 0.4"
 			}
 		},
+		"node_modules/html-encoding-sniffer": {
+			"version": "4.0.0",
+			"resolved": "https://registry.npmjs.org/html-encoding-sniffer/-/html-encoding-sniffer-4.0.0.tgz",
+			"integrity": "sha512-Y22oTqIU4uuPgEemfz7NDJz6OeKf12Lsu+QC+s3BVpda64lTiMYCyGwg5ki4vFxkMwQdeZDl2adZoqUgdFuTgQ==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"whatwg-encoding": "^3.1.1"
+			},
+			"engines": {
+				"node": ">=18"
+			}
+		},
 		"node_modules/html-escaper": {
 			"version": "2.0.2",
 			"resolved": "https://registry.npmjs.org/html-escaper/-/html-escaper-2.0.2.tgz",
@@ -2127,6 +2201,47 @@
 			"dev": true,
 			"license": "MIT"
 		},
+		"node_modules/http-proxy-agent": {
+			"version": "7.0.2",
+			"resolved": "https://registry.npmjs.org/http-proxy-agent/-/http-proxy-agent-7.0.2.tgz",
+			"integrity": "sha512-T1gkAiYYDWYx3V5Bmyu7HcfcvL7mUrTWiM6yOfa3PIphViJ/gFPbvidQ+veqSOHci/PxBcDabeUNCzpOODJZig==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"agent-base": "^7.1.0",
+				"debug": "^4.3.4"
+			},
+			"engines": {
+				"node": ">= 14"
+			}
+		},
+		"node_modules/https-proxy-agent": {
+			"version": "7.0.5",
+			"resolved": "https://registry.npmjs.org/https-proxy-agent/-/https-proxy-agent-7.0.5.tgz",
+			"integrity": "sha512-1e4Wqeblerz+tMKPIq2EMGiiWW1dIjZOksyHWSUm1rmuvw/how9hBHZ38lAGj5ID4Ik6EdkOw7NmWPy6LAwalw==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"agent-base": "^7.0.2",
+				"debug": "4"
+			},
+			"engines": {
+				"node": ">= 14"
+			}
+		},
+		"node_modules/iconv-lite": {
+			"version": "0.6.3",
+			"resolved": "https://registry.npmjs.org/iconv-lite/-/iconv-lite-0.6.3.tgz",
+			"integrity": "sha512-4fCk79wshMdzMp2rH06qWrJE4iolqLhCUH+OiuIgU++RB0+94NlDL81atO7GX55uUKueo0txHNtvEyI6D7WdMw==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"safer-buffer": ">= 2.1.2 < 3.0.0"
+			},
+			"engines": {
+				"node": ">=0.10.0"
+			}
+		},
 		"node_modules/ignore": {
 			"version": "5.3.2",
 			"resolved": "https://registry.npmjs.org/ignore/-/ignore-5.3.2.tgz",
@@ -2247,6 +2362,13 @@
 				"node": ">=0.12.0"
 			}
 		},
+		"node_modules/is-potential-custom-element-name": {
+			"version": "1.0.1",
+			"resolved": "https://registry.npmjs.org/is-potential-custom-element-name/-/is-potential-custom-element-name-1.0.1.tgz",
+			"integrity": "sha512-bCYeRA2rVibKZd+s2625gGnGF/t7DSqDs4dP7CrLA1m7jKWz6pps0LpYLJN8Q64HtmPKJ1hrN3nzPNKFEKOUiQ==",
+			"dev": true,
+			"license": "MIT"
+		},
 		"node_modules/is-reference": {
 			"version": "3.0.2",
 			"resolved": "https://registry.npmjs.org/is-reference/-/is-reference-3.0.2.tgz",
@@ -2357,6 +2479,47 @@
 				"js-yaml": "bin/js-yaml.js"
 			}
 		},
+		"node_modules/jsdom": {
+			"version": "25.0.1",
+			"resolved": "https://registry.npmjs.org/jsdom/-/jsdom-25.0.1.tgz",
+			"integrity": "sha512-8i7LzZj7BF8uplX+ZyOlIz86V6TAsSs+np6m1kpW9u0JWi4z/1t+FzcK1aek+ybTnAC4KhBL4uXCNT0wcUIeCw==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"cssstyle": "^4.1.0",
+				"data-urls": "^5.0.0",
+				"decimal.js": "^10.4.3",
+				"form-data": "^4.0.0",
+				"html-encoding-sniffer": "^4.0.0",
+				"http-proxy-agent": "^7.0.2",
+				"https-proxy-agent": "^7.0.5",
+				"is-potential-custom-element-name": "^1.0.1",
+				"nwsapi": "^2.2.12",
+				"parse5": "^7.1.2",
+				"rrweb-cssom": "^0.7.1",
+				"saxes": "^6.0.0",
+				"symbol-tree": "^3.2.4",
+				"tough-cookie": "^5.0.0",
+				"w3c-xmlserializer": "^5.0.0",
+				"webidl-conversions": "^7.0.0",
+				"whatwg-encoding": "^3.1.1",
+				"whatwg-mimetype": "^4.0.0",
+				"whatwg-url": "^14.0.0",
+				"ws": "^8.18.0",
+				"xml-name-validator": "^5.0.0"
+			},
+			"engines": {
+				"node": ">=18"
+			},
+			"peerDependencies": {
+				"canvas": "^2.11.2"
+			},
+			"peerDependenciesMeta": {
+				"canvas": {
+					"optional": true
+				}
+			}
+		},
 		"node_modules/json-buffer": {
 			"version": "3.0.1",
 			"resolved": "https://registry.npmjs.org/json-buffer/-/json-buffer-3.0.1.tgz",
@@ -2700,6 +2863,13 @@
 				"node": ">=0.10.0"
 			}
 		},
+		"node_modules/nwsapi": {
+			"version": "2.2.13",
+			"resolved": "https://registry.npmjs.org/nwsapi/-/nwsapi-2.2.13.tgz",
+			"integrity": "sha512-cTGB9ptp9dY9A5VbMSe7fQBcl/tt22Vcqdq8+eN93rblOuE0aCFu4aZ2vMwct/2t+lFnosm8RkQW1I0Omb1UtQ==",
+			"dev": true,
+			"license": "MIT"
+		},
 		"node_modules/object-assign": {
 			"version": "4.1.1",
 			"resolved": "https://registry.npmjs.org/object-assign/-/object-assign-4.1.1.tgz",
@@ -2790,6 +2960,19 @@
 				"node": ">=6"
 			}
 		},
+		"node_modules/parse5": {
+			"version": "7.2.1",
+			"resolved": "https://registry.npmjs.org/parse5/-/parse5-7.2.1.tgz",
+			"integrity": "sha512-BuBYQYlv1ckiPdQi/ohiivi9Sagc9JG+Ozs0r7b/0iK3sKmrb0b9FdWdBbOdx6hBCM/F9Ir82ofnBhtZOjCRPQ==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"entities": "^4.5.0"
+			},
+			"funding": {
+				"url": "https://github.com/inikulin/parse5?sponsor=1"
+			}
+		},
 		"node_modules/path-exists": {
 			"version": "4.0.0",
 			"resolved": "https://registry.npmjs.org/path-exists/-/path-exists-4.0.0.tgz",
@@ -3251,6 +3434,13 @@
 				"fsevents": "~2.3.2"
 			}
 		},
+		"node_modules/rrweb-cssom": {
+			"version": "0.7.1",
+			"resolved": "https://registry.npmjs.org/rrweb-cssom/-/rrweb-cssom-0.7.1.tgz",
+			"integrity": "sha512-TrEMa7JGdVm0UThDJSx7ddw5nVm3UJS9o9CCIZ72B1vSyEZoziDqBYP3XIoi/12lKrJR8rE3jeFHMok2F/Mnsg==",
+			"dev": true,
+			"license": "MIT"
+		},
 		"node_modules/run-parallel": {
 			"version": "1.2.0",
 			"resolved": "https://registry.npmjs.org/run-parallel/-/run-parallel-1.2.0.tgz",
@@ -3288,6 +3478,26 @@
 				"node": ">=6"
 			}
 		},
+		"node_modules/safer-buffer": {
+			"version": "2.1.2",
+			"resolved": "https://registry.npmjs.org/safer-buffer/-/safer-buffer-2.1.2.tgz",
+			"integrity": "sha512-YZo3K82SD7Riyi0E1EQPojLz7kpepnSQI9IyPbHHg1XXXevb5dJI7tpyN2ADxGcQbHG7vcyRHk0cbwqcQriUtg==",
+			"dev": true,
+			"license": "MIT"
+		},
+		"node_modules/saxes": {
+			"version": "6.0.0",
+			"resolved": "https://registry.npmjs.org/saxes/-/saxes-6.0.0.tgz",
+			"integrity": "sha512-xAg7SOnEhrm5zI3puOOKyy1OMcMlIJZYNJY7xLBwSze0UjhPLnWfj2GF2EpT0jmzaJKIWKHLsaSSajf35bcYnA==",
+			"dev": true,
+			"license": "ISC",
+			"dependencies": {
+				"xmlchars": "^2.2.0"
+			},
+			"engines": {
+				"node": ">=v12.22.7"
+			}
+		},
 		"node_modules/semver": {
 			"version": "7.6.3",
 			"resolved": "https://registry.npmjs.org/semver/-/semver-7.6.3.tgz",
@@ -3657,6 +3867,13 @@
 				"url": "https://opencollective.com/eslint"
 			}
 		},
+		"node_modules/symbol-tree": {
+			"version": "3.2.4",
+			"resolved": "https://registry.npmjs.org/symbol-tree/-/symbol-tree-3.2.4.tgz",
+			"integrity": "sha512-9QNk5KwDF+Bvz+PyObkmSYjI5ksVUYtjW7AU22r2NKcfLJcXp96hkDWU3+XndOsUb+AQ9QhfzfCT2O+CNWT5Tw==",
+			"dev": true,
+			"license": "MIT"
+		},
 		"node_modules/tailwindcss": {
 			"version": "3.4.14",
 			"resolved": "https://registry.npmjs.org/tailwindcss/-/tailwindcss-3.4.14.tgz",
@@ -3883,6 +4100,26 @@
 				"node": ">=14.0.0"
 			}
 		},
+		"node_modules/tldts": {
+			"version": "6.1.60",
+			"resolved": "https://registry.npmjs.org/tldts/-/tldts-6.1.60.tgz",
+			"integrity": "sha512-TYVHm7G9NCnhgqOsFalbX6MG1Po5F4efF+tLfoeiOGQq48Oqgwcgz8upY2R1BHWa4aDrj28RYx0dkYJ63qCFMg==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"tldts-core": "^6.1.60"
+			},
+			"bin": {
+				"tldts": "bin/cli.js"
+			}
+		},
+		"node_modules/tldts-core": {
+			"version": "6.1.60",
+			"resolved": "https://registry.npmjs.org/tldts-core/-/tldts-core-6.1.60.tgz",
+			"integrity": "sha512-XHjoxak8SFQnHnmYHb3PcnW5TZ+9ErLZemZei3azuIRhQLw4IExsVbL3VZJdHcLeNaXq6NqawgpDPpjBOg4B5g==",
+			"dev": true,
+			"license": "MIT"
+		},
 		"node_modules/to-regex-range": {
 			"version": "5.0.1",
 			"resolved": "https://registry.npmjs.org/to-regex-range/-/to-regex-range-5.0.1.tgz",
@@ -3906,6 +4143,32 @@
 				"node": ">=6"
 			}
 		},
+		"node_modules/tough-cookie": {
+			"version": "5.0.0",
+			"resolved": "https://registry.npmjs.org/tough-cookie/-/tough-cookie-5.0.0.tgz",
+			"integrity": "sha512-FRKsF7cz96xIIeMZ82ehjC3xW2E+O2+v11udrDYewUbszngYhsGa8z6YUMMzO9QJZzzyd0nGGXnML/TReX6W8Q==",
+			"dev": true,
+			"license": "BSD-3-Clause",
+			"dependencies": {
+				"tldts": "^6.1.32"
+			},
+			"engines": {
+				"node": ">=16"
+			}
+		},
+		"node_modules/tr46": {
+			"version": "5.0.0",
+			"resolved": "https://registry.npmjs.org/tr46/-/tr46-5.0.0.tgz",
+			"integrity": "sha512-tk2G5R2KRwBd+ZN0zaEXpmzdKyOYksXwywulIX95MBODjSzMIuQnQ3m8JxgbhnL1LeVo7lqQKsYa1O3Htl7K5g==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"punycode": "^2.3.1"
+			},
+			"engines": {
+				"node": ">=18"
+			}
+		},
 		"node_modules/ts-interface-checker": {
 			"version": "0.1.13",
 			"resolved": "https://registry.npmjs.org/ts-interface-checker/-/ts-interface-checker-0.1.13.tgz",
@@ -4141,6 +4404,66 @@
 				}
 			}
 		},
+		"node_modules/w3c-xmlserializer": {
+			"version": "5.0.0",
+			"resolved": "https://registry.npmjs.org/w3c-xmlserializer/-/w3c-xmlserializer-5.0.0.tgz",
+			"integrity": "sha512-o8qghlI8NZHU1lLPrpi2+Uq7abh4GGPpYANlalzWxyWteJOCsr/P+oPBA49TOLu5FTZO4d3F9MnWJfiMo4BkmA==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"xml-name-validator": "^5.0.0"
+			},
+			"engines": {
+				"node": ">=18"
+			}
+		},
+		"node_modules/webidl-conversions": {
+			"version": "7.0.0",
+			"resolved": "https://registry.npmjs.org/webidl-conversions/-/webidl-conversions-7.0.0.tgz",
+			"integrity": "sha512-VwddBukDzu71offAQR975unBIGqfKZpM+8ZX6ySk8nYhVoo5CYaZyzt3YBvYtRtO+aoGlqxPg/B87NGVZ/fu6g==",
+			"dev": true,
+			"license": "BSD-2-Clause",
+			"engines": {
+				"node": ">=12"
+			}
+		},
+		"node_modules/whatwg-encoding": {
+			"version": "3.1.1",
+			"resolved": "https://registry.npmjs.org/whatwg-encoding/-/whatwg-encoding-3.1.1.tgz",
+			"integrity": "sha512-6qN4hJdMwfYBtE3YBTTHhoeuUrDBPZmbQaxWAqSALV/MeEnR5z1xd8UKud2RAkFoPkmB+hli1TZSnyi84xz1vQ==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"iconv-lite": "0.6.3"
+			},
+			"engines": {
+				"node": ">=18"
+			}
+		},
+		"node_modules/whatwg-mimetype": {
+			"version": "4.0.0",
+			"resolved": "https://registry.npmjs.org/whatwg-mimetype/-/whatwg-mimetype-4.0.0.tgz",
+			"integrity": "sha512-QaKxh0eNIi2mE9p2vEdzfagOKHCcj1pJ56EEHGQOVxp8r9/iszLUUV7v89x9O1p/T+NlTM5W7jW6+cz4Fq1YVg==",
+			"dev": true,
+			"license": "MIT",
+			"engines": {
+				"node": ">=18"
+			}
+		},
+		"node_modules/whatwg-url": {
+			"version": "14.0.0",
+			"resolved": "https://registry.npmjs.org/whatwg-url/-/whatwg-url-14.0.0.tgz",
+			"integrity": "sha512-1lfMEm2IEr7RIV+f4lUNPOqfFL+pO+Xw3fJSqmjX9AbXcXcYOkCe1P6+9VBZB6n94af16NfZf+sSk0JCBZC9aw==",
+			"dev": true,
+			"license": "MIT",
+			"dependencies": {
+				"tr46": "^5.0.0",
+				"webidl-conversions": "^7.0.0"
+			},
+			"engines": {
+				"node": ">=18"
+			}
+		},
 		"node_modules/which": {
 			"version": "2.0.2",
 			"resolved": "https://registry.npmjs.org/which/-/which-2.0.2.tgz",
@@ -4279,6 +4602,45 @@
 				"url": "https://github.com/chalk/ansi-styles?sponsor=1"
 			}
 		},
+		"node_modules/ws": {
+			"version": "8.18.0",
+			"resolved": "https://registry.npmjs.org/ws/-/ws-8.18.0.tgz",
+			"integrity": "sha512-8VbfWfHLbbwu3+N6OKsOMpBdT4kXPDDB9cJk2bJ6mh9ucxdlnNvH1e+roYkKmN9Nxw2yjz7VzeO9oOz2zJ04Pw==",
+			"dev": true,
+			"license": "MIT",
+			"engines": {
+				"node": ">=10.0.0"
+			},
+			"peerDependencies": {
+				"bufferutil": "^4.0.1",
+				"utf-8-validate": ">=5.0.2"
+			},
+			"peerDependenciesMeta": {
+				"bufferutil": {
+					"optional": true
+				},
+				"utf-8-validate": {
+					"optional": true
+				}
+			}
+		},
+		"node_modules/xml-name-validator": {
+			"version": "5.0.0",
+			"resolved": "https://registry.npmjs.org/xml-name-validator/-/xml-name-validator-5.0.0.tgz",
+			"integrity": "sha512-EvGK8EJ3DhaHfbRlETOWAS5pO9MZITeauHKJyb8wyajUfQUenkIg2MvLDTZ4T/TgIcm3HU0TFBgWWboAZ30UHg==",
+			"dev": true,
+			"license": "Apache-2.0",
+			"engines": {
+				"node": ">=18"
+			}
+		},
+		"node_modules/xmlchars": {
+			"version": "2.2.0",
+			"resolved": "https://registry.npmjs.org/xmlchars/-/xmlchars-2.2.0.tgz",
+			"integrity": "sha512-JZnDKK8B0RCDw84FNdDAIpZK+JuJw+s7Lz8nksI7SIuU3UXJJslUthsi+uWBUYOwPFwW7W7PRLRfUKpxjtjFCw==",
+			"dev": true,
+			"license": "MIT"
+		},
 		"node_modules/yaml": {
 			"version": "1.10.2",
 			"resolved": "https://registry.npmjs.org/yaml/-/yaml-1.10.2.tgz",
diff --git a/package.json b/package.json
index 7e36802..18a80f4 100644
--- a/package.json
+++ b/package.json
@@ -29,6 +29,7 @@
 		"eslint-config-prettier": "^9.1.0",
 		"eslint-plugin-svelte": "^2.46.0",
 		"globals": "^15.11.0",
+		"jsdom": "^25.0.1",
 		"postcss": "^8.4.47",
 		"prettier": "^3.3.3",
 		"prettier-plugin-svelte": "^3.2.7",
diff --git a/ui/tests/lib/components/ChangePassword.svelte.test.js b/ui/tests/lib/components/ChangePassword.svelte.test.js
new file mode 100644
index 0000000..6e0e652
--- /dev/null
+++ b/ui/tests/lib/components/ChangePassword.svelte.test.js
@@ -0,0 +1,28 @@
+// async onsubmit(event)
+//
+// Example:
+/* */
+import { flushSync, mount, unmount } from 'svelte';
+import { expect, test } from 'vitest';
+import ChangePassword from '$lib/components/ChangePassword.svelte';
+
+
+test('ChangePassword', () => {
+	// Instantiate the component using Svelte's `mount` API
+	const component = mount(ChangePassword, {
+		target: document.body, // `document` exists because of jsdom
+		props: { initial: 0 }
+	});
+
+	expect(document.body.innerHTML).toBe('<button>0</button>');
+
+	// Click the button, then flush the changes so you can synchronously write expectations
+	document.body.querySelector('button').click();
+	flushSync();
+
+	expect(document.body.innerHTML).toBe('<button>1</button>');
+
+	// Remove the component from the DOM
+	unmount(component);
+});
+/* */
diff --git a/ui/tests/lib/components/CreateChannelForm.svelte.test.js b/ui/tests/lib/components/CreateChannelForm.svelte.test.js
new file mode 100644
index 0000000..0380e01
--- /dev/null
+++ b/ui/tests/lib/components/CreateChannelForm.svelte.test.js
@@ -0,0 +1 @@
+// async handleSubmit(event)
diff --git a/ui/tests/lib/components/Invite.svelte.test.js b/ui/tests/lib/components/Invite.svelte.test.js
new file mode 100644
index 0000000..e4383cc
--- /dev/null
+++ b/ui/tests/lib/components/Invite.svelte.test.js
@@ -0,0 +1 @@
+// async onsubmit(event)
diff --git a/ui/tests/lib/components/LogOut.svelte.test.js b/ui/tests/lib/components/LogOut.svelte.test.js
new file mode 100644
index 0000000..e4383cc
--- /dev/null
+++ b/ui/tests/lib/components/LogOut.svelte.test.js
@@ -0,0 +1 @@
+// async onsubmit(event)
diff --git a/ui/tests/lib/components/Message.svelte.test.js b/ui/tests/lib/components/Message.svelte.test.js
new file mode 100644
index 0000000..d29a371
--- /dev/null
+++ b/ui/tests/lib/components/Message.svelte.test.js
@@ -0,0 +1,2 @@
+// onDelete(event)
+// onmouseleave()
diff --git a/ui/tests/lib/components/MessageInput.svelte.test.js b/ui/tests/lib/components/MessageInput.svelte.test.js
new file mode 100644
index 0000000..7c7f9aa
--- /dev/null
+++ b/ui/tests/lib/components/MessageInput.svelte.test.js
@@ -0,0 +1,2 @@
+// async onSubmit(event)
+// onKeyDown(event)
diff --git a/vite.config.js b/vite.config.js
index 61ba597..8d6866d 100644
--- a/vite.config.js
+++ b/vite.config.js
@@ -3,6 +3,12 @@ import { defineConfig } from 'vite';
 
 export default defineConfig({
   plugins: [sveltekit()],
+  test: {
+    // If you are testing components client-side, you need to setup a DOM environment.
+    // If not all your files should have this environment, you can use a
+    // `// @vitest-environment jsdom` comment at the top of the test files instead.
+    environment: 'jsdom'
+  },
   // Tell Vitest to use the `browser` entry points in `package.json` files,
   // even though it's running in Node
 	resolve: process.env.VITEST ? { conditions: ['browser'] } : undefined,
-- 
cgit v1.2.3


From 3a89459f8b48449b341469776a37d8b225e956f2 Mon Sep 17 00:00:00 2001
From: Kit La Touche <kit@transneptune.net>
Date: Thu, 14 Nov 2024 23:28:19 -0500
Subject: Stub in tests

---
 .../lib/components/ChangePassword.svelte.test.js   | 45 +++++++++++++---------
 .../components/CreateChannelForm.svelte.test.js    |  7 ++++
 ui/tests/lib/components/Invite.svelte.test.js      |  7 ++++
 ui/tests/lib/components/LogOut.svelte.test.js      |  7 ++++
 ui/tests/lib/components/Message.svelte.test.js     |  7 ++++
 .../lib/components/MessageInput.svelte.test.js     |  7 ++++
 6 files changed, 61 insertions(+), 19 deletions(-)

(limited to 'ui')

diff --git a/ui/tests/lib/components/ChangePassword.svelte.test.js b/ui/tests/lib/components/ChangePassword.svelte.test.js
index 6e0e652..09aa992 100644
--- a/ui/tests/lib/components/ChangePassword.svelte.test.js
+++ b/ui/tests/lib/components/ChangePassword.svelte.test.js
@@ -1,28 +1,35 @@
-// async onsubmit(event)
-//
-// Example:
-/* */
 import { flushSync, mount, unmount } from 'svelte';
-import { expect, test } from 'vitest';
+import { afterEach, describe, beforeEach, expect, test, vi } from 'vitest';
 import ChangePassword from '$lib/components/ChangePassword.svelte';
+import axios from 'axios';
 
+vi.mock('axios');
 
-test('ChangePassword', () => {
-	// Instantiate the component using Svelte's `mount` API
-	const component = mount(ChangePassword, {
-		target: document.body, // `document` exists because of jsdom
-		props: { initial: 0 }
-	});
+describe('ChangePassword', () => {
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
 
-	expect(document.body.innerHTML).toBe('<button>0</button>');
+  test('onsubmit happy path', async () => {
+    axios.post.mockResolvedValue({ status: 200 });
 
-	// Click the button, then flush the changes so you can synchronously write expectations
-	document.body.querySelector('button').click();
-	flushSync();
+    // Instantiate the component using Svelte's `mount` API
+    const component = mount(ChangePassword, {
+      target: document.body, // `document` exists because of jsdom
+    });
+    flushSync();
 
-	expect(document.body.innerHTML).toBe('<button>1</button>');
+    // Set value in all three inputs at once:
+    const inputs = document.body.querySelectorAll('input[type=password]');
+    inputs.value = 'pass';
+    // Click the button, then flush the changes so you can synchronously write expectations
+    document.body.querySelector('button[type=submit]').click();
+    flushSync();
 
-	// Remove the component from the DOM
-	unmount(component);
+    // expect(axios.post).toHaveBeenCalledWith('/password', { password: 'pass', to: 'pass' });
+    expect(Array.from(inputs.values()).map((i) => i.value)).toEqual(['', '', '']);
+
+    // Remove the component from the DOM
+    unmount(component);
+  });
 });
-/* */
diff --git a/ui/tests/lib/components/CreateChannelForm.svelte.test.js b/ui/tests/lib/components/CreateChannelForm.svelte.test.js
index 0380e01..31f7462 100644
--- a/ui/tests/lib/components/CreateChannelForm.svelte.test.js
+++ b/ui/tests/lib/components/CreateChannelForm.svelte.test.js
@@ -1 +1,8 @@
 // async handleSubmit(event)
+import { describe, expect, test } from 'vitest';
+
+describe('CreateChannelForm', () => {
+  test('stub', async () => {
+    expect(true).toBeTruthy();
+  });
+});
diff --git a/ui/tests/lib/components/Invite.svelte.test.js b/ui/tests/lib/components/Invite.svelte.test.js
index e4383cc..5d5742e 100644
--- a/ui/tests/lib/components/Invite.svelte.test.js
+++ b/ui/tests/lib/components/Invite.svelte.test.js
@@ -1 +1,8 @@
 // async onsubmit(event)
+import { describe, expect, test } from 'vitest';
+
+describe('Invite', () => {
+  test('stub', async () => {
+    expect(true).toBeTruthy();
+  });
+});
diff --git a/ui/tests/lib/components/LogOut.svelte.test.js b/ui/tests/lib/components/LogOut.svelte.test.js
index e4383cc..65d78a9 100644
--- a/ui/tests/lib/components/LogOut.svelte.test.js
+++ b/ui/tests/lib/components/LogOut.svelte.test.js
@@ -1 +1,8 @@
 // async onsubmit(event)
+import { describe, expect, test } from 'vitest';
+
+describe('LogOut', () => {
+  test('stub', async () => {
+    expect(true).toBeTruthy();
+  });
+});
diff --git a/ui/tests/lib/components/Message.svelte.test.js b/ui/tests/lib/components/Message.svelte.test.js
index d29a371..2ca0b85 100644
--- a/ui/tests/lib/components/Message.svelte.test.js
+++ b/ui/tests/lib/components/Message.svelte.test.js
@@ -1,2 +1,9 @@
 // onDelete(event)
 // onmouseleave()
+import { describe, expect, test } from 'vitest';
+
+describe('Message', () => {
+  test('stub', async () => {
+    expect(true).toBeTruthy();
+  });
+});
diff --git a/ui/tests/lib/components/MessageInput.svelte.test.js b/ui/tests/lib/components/MessageInput.svelte.test.js
index 7c7f9aa..597f0c3 100644
--- a/ui/tests/lib/components/MessageInput.svelte.test.js
+++ b/ui/tests/lib/components/MessageInput.svelte.test.js
@@ -1,2 +1,9 @@
 // async onSubmit(event)
 // onKeyDown(event)
+import { describe, expect, test } from 'vitest';
+
+describe('MessageInput', () => {
+  test('stub', async () => {
+    expect(true).toBeTruthy();
+  });
+});
-- 
cgit v1.2.3


From 122ecb5769e20173c99033ade0057d38e12cb384 Mon Sep 17 00:00:00 2001
From: Owen Jacobson <owen@grimoire.ca>
Date: Fri, 15 Nov 2024 01:18:41 -0500
Subject: HTML-ify message bodies once, on receipt, and not every time the
 message is rendered.

~16% of the `hi development` channel's rendering time was taken up on this.
---
 ui/lib/components/Message.svelte    | 6 +-----
 ui/lib/components/MessageRun.svelte | 4 ++--
 ui/lib/store/messages.svelte.js     | 6 +++++-
 3 files changed, 8 insertions(+), 8 deletions(-)

(limited to 'ui')

diff --git a/ui/lib/components/Message.svelte b/ui/lib/components/Message.svelte
index 0c8eeec..fddeecd 100644
--- a/ui/lib/components/Message.svelte
+++ b/ui/lib/components/Message.svelte
@@ -1,15 +1,11 @@
 <script>
-  import { marked } from 'marked';
-  import DOMPurify from 'dompurify';
-
   import { deleteMessage } from '$lib/apiServer';
 
   function scroll(message) {
     message.scrollIntoView();
   }
 
-  let { id, at, body, editable = false } = $props();
-  let renderedBody = $derived(DOMPurify.sanitize(marked.parse(body, { breaks: true })));
+  let { id, at, body, renderedBody, editable = false } = $props();
   let deleteArmed = $state(false);
 
   function onDelete(event) {
diff --git a/ui/lib/components/MessageRun.svelte b/ui/lib/components/MessageRun.svelte
index 83a82a9..39ee155 100644
--- a/ui/lib/components/MessageRun.svelte
+++ b/ui/lib/components/MessageRun.svelte
@@ -16,7 +16,7 @@
   <span class="chip variant-soft sticky top-o left-0">
     @{name}:
   </span>
-  {#each messages as { id, at, body }}
-    <Message {id} {at} {body} editable={ownMessage} />
+  {#each messages as { id, at, body, renderedBody }}
+    <Message {id} {at} {body} {renderedBody} editable={ownMessage} />
   {/each}
 </div>
diff --git a/ui/lib/store/messages.svelte.js b/ui/lib/store/messages.svelte.js
index c0db71b..0ceba54 100644
--- a/ui/lib/store/messages.svelte.js
+++ b/ui/lib/store/messages.svelte.js
@@ -1,3 +1,6 @@
+import { marked } from 'marked';
+import DOMPurify from 'dompurify';
+
 const RUN_COALESCE_MAX_INTERVAL = 10 /* min */ * 60 /* sec */ * 1000; /* ms */
 
 export class Messages {
@@ -9,7 +12,8 @@ export class Messages {
 
   addMessage(channel, id, { at, sender, body }) {
     let parsedAt = new Date(at);
-    const message = { id, at: parsedAt, body };
+    let renderedBody = DOMPurify.sanitize(marked.parse(body, { breaks: true }));
+    const message = { id, at: parsedAt, body, renderedBody };
 
     // You might be thinking, can't this be
     //
-- 
cgit v1.2.3


From 8a7cc9f8859c14b60da870cee199110e71e0f5da Mon Sep 17 00:00:00 2001
From: Kit La Touche <kit@transneptune.net>
Date: Fri, 15 Nov 2024 22:37:39 -0500
Subject: Ratchet up Component test coverage

Turns out a number of our components are a pain to run via a mounting
test, but Svelte does suggest that this is the dispreferred way to test
anyway. Using use:, triggering navigation after logout, and
scrollIntoView all make the headless node test process with the janky
in-memory mount have a Very Bad Day.

But this is still progress! We will press on.
---
 .../lib/components/ChangePassword.svelte.test.js   | 41 ++++++++++++-----
 .../components/CreateChannelForm.svelte.test.js    | 52 ++++++++++++++++++++--
 ui/tests/lib/components/Invite.svelte.test.js      |  8 ----
 ui/tests/lib/components/LogOut.svelte.test.js      |  8 ----
 ui/tests/lib/components/Message.svelte.test.js     |  9 ----
 .../lib/components/MessageInput.svelte.test.js     | 49 +++++++++++++++++---
 vite.config.js                                     | 17 ++++++-
 7 files changed, 137 insertions(+), 47 deletions(-)
 delete mode 100644 ui/tests/lib/components/Invite.svelte.test.js
 delete mode 100644 ui/tests/lib/components/LogOut.svelte.test.js
 delete mode 100644 ui/tests/lib/components/Message.svelte.test.js

(limited to 'ui')

diff --git a/ui/tests/lib/components/ChangePassword.svelte.test.js b/ui/tests/lib/components/ChangePassword.svelte.test.js
index 09aa992..5773eae 100644
--- a/ui/tests/lib/components/ChangePassword.svelte.test.js
+++ b/ui/tests/lib/components/ChangePassword.svelte.test.js
@@ -1,24 +1,44 @@
 import { flushSync, mount, unmount } from 'svelte';
-import { afterEach, describe, beforeEach, expect, test, vi } from 'vitest';
+import { afterEach, beforeEach, describe, expect, test, vi } from 'vitest';
 import ChangePassword from '$lib/components/ChangePassword.svelte';
 import axios from 'axios';
 
-vi.mock('axios');
+let component;
+
+let mocks = vi.hoisted(() => ({
+  post: vi.fn(),
+}));
 
 describe('ChangePassword', () => {
-  afterEach(() => {
-    vi.restoreAllMocks();
-  });
+  beforeEach(() => {
+    vi.mock('axios', async (importActual) => {
+      const actual = await importActual();
 
-  test('onsubmit happy path', async () => {
-    axios.post.mockResolvedValue({ status: 200 });
+      const mockAxios = {
+        default: {
+          ...actual.default,
+          create: vi.fn(() => ({
+            ...actual.default.create(),
+            post: mocks.post,
+          })),
+        },
+      };
 
-    // Instantiate the component using Svelte's `mount` API
-    const component = mount(ChangePassword, {
+      return mockAxios;
+    });
+    mocks.post.mockResolvedValue({ status: 200 });
+
+    component = mount(ChangePassword, {
       target: document.body, // `document` exists because of jsdom
     });
     flushSync();
+  });
 
+  afterEach(() => {
+    unmount(component);
+  });
+
+  test('onsubmit happy path', async () => {
     // Set value in all three inputs at once:
     const inputs = document.body.querySelectorAll('input[type=password]');
     inputs.value = 'pass';
@@ -28,8 +48,5 @@ describe('ChangePassword', () => {
 
     // expect(axios.post).toHaveBeenCalledWith('/password', { password: 'pass', to: 'pass' });
     expect(Array.from(inputs.values()).map((i) => i.value)).toEqual(['', '', '']);
-
-    // Remove the component from the DOM
-    unmount(component);
   });
 });
diff --git a/ui/tests/lib/components/CreateChannelForm.svelte.test.js b/ui/tests/lib/components/CreateChannelForm.svelte.test.js
index 31f7462..c1adba5 100644
--- a/ui/tests/lib/components/CreateChannelForm.svelte.test.js
+++ b/ui/tests/lib/components/CreateChannelForm.svelte.test.js
@@ -1,8 +1,52 @@
-// async handleSubmit(event)
-import { describe, expect, test } from 'vitest';
+import { flushSync, mount, unmount } from 'svelte';
+import { afterEach, beforeEach, describe, expect, test, vi } from 'vitest';
+import CreateChannelForm from '$lib/components/CreateChannelForm.svelte';
+import axios from 'axios';
+
+let component;
+
+let mocks = vi.hoisted(() => ({
+  post: vi.fn(),
+}));
 
 describe('CreateChannelForm', () => {
-  test('stub', async () => {
-    expect(true).toBeTruthy();
+  beforeEach(() => {
+    vi.mock('axios', async (importActual) => {
+      const actual = await importActual();
+
+      const mockAxios = {
+        default: {
+          ...actual.default,
+          create: vi.fn(() => ({
+            ...actual.default.create(),
+            post: mocks.post,
+          })),
+        },
+      };
+
+      return mockAxios;
+    });
+    mocks.post.mockResolvedValue({ status: 200 });
+
+    component = mount(CreateChannelForm, {
+      target: document.body, // `document` exists because of jsdom
+    });
+    flushSync();
+  });
+
+  afterEach(() => {
+    unmount(component);
+  });
+
+  test('onsubmit happy path', async () => {
+    // Set value on the one input this should match:
+    const inputs = document.body.querySelectorAll('input[type=text]');
+    inputs.value = 'channel name';
+    // Click the button, then flush the changes so you can synchronously write expectations
+    document.body.querySelector('button[type=submit]').click();
+    flushSync();
+
+    expect(mocks.post).toHaveBeenCalled();
+    expect(Array.from(inputs.values()).map((i) => i.value)).toEqual(['']);
   });
 });
diff --git a/ui/tests/lib/components/Invite.svelte.test.js b/ui/tests/lib/components/Invite.svelte.test.js
deleted file mode 100644
index 5d5742e..0000000
--- a/ui/tests/lib/components/Invite.svelte.test.js
+++ /dev/null
@@ -1,8 +0,0 @@
-// async onsubmit(event)
-import { describe, expect, test } from 'vitest';
-
-describe('Invite', () => {
-  test('stub', async () => {
-    expect(true).toBeTruthy();
-  });
-});
diff --git a/ui/tests/lib/components/LogOut.svelte.test.js b/ui/tests/lib/components/LogOut.svelte.test.js
deleted file mode 100644
index 65d78a9..0000000
--- a/ui/tests/lib/components/LogOut.svelte.test.js
+++ /dev/null
@@ -1,8 +0,0 @@
-// async onsubmit(event)
-import { describe, expect, test } from 'vitest';
-
-describe('LogOut', () => {
-  test('stub', async () => {
-    expect(true).toBeTruthy();
-  });
-});
diff --git a/ui/tests/lib/components/Message.svelte.test.js b/ui/tests/lib/components/Message.svelte.test.js
deleted file mode 100644
index 2ca0b85..0000000
--- a/ui/tests/lib/components/Message.svelte.test.js
+++ /dev/null
@@ -1,9 +0,0 @@
-// onDelete(event)
-// onmouseleave()
-import { describe, expect, test } from 'vitest';
-
-describe('Message', () => {
-  test('stub', async () => {
-    expect(true).toBeTruthy();
-  });
-});
diff --git a/ui/tests/lib/components/MessageInput.svelte.test.js b/ui/tests/lib/components/MessageInput.svelte.test.js
index 597f0c3..d90026c 100644
--- a/ui/tests/lib/components/MessageInput.svelte.test.js
+++ b/ui/tests/lib/components/MessageInput.svelte.test.js
@@ -1,9 +1,48 @@
-// async onSubmit(event)
-// onKeyDown(event)
-import { describe, expect, test } from 'vitest';
+import { flushSync, mount, unmount } from 'svelte';
+import { afterEach, beforeEach, describe, expect, test, vi } from 'vitest';
+import MessageInput from '$lib/components/MessageInput.svelte';
+import axios from 'axios';
+
+let component;
+
+let mocks = vi.hoisted(() => ({
+  post: vi.fn(),
+}));
 
 describe('MessageInput', () => {
-  test('stub', async () => {
-    expect(true).toBeTruthy();
+  beforeEach(() => {
+    vi.mock('axios', async (importActual) => {
+      const actual = await importActual();
+
+      const mockAxios = {
+        default: {
+          ...actual.default,
+          create: vi.fn(() => ({
+            ...actual.default.create(),
+            post: mocks.post,
+          })),
+        },
+      };
+
+      return mockAxios;
+    });
+    mocks.post.mockResolvedValue({ status: 200 });
+
+    component = mount(MessageInput, {
+      target: document.body, // `document` exists because of jsdom
+    });
+    flushSync();
+  });
+
+  afterEach(() => {
+    unmount(component);
+  });
+
+  test('onsubmit happy path', async () => {
+    // Click the button, then flush the changes so you can synchronously write expectations
+    document.body.querySelector('button[type=submit]').click();
+    flushSync();
+
+    expect(mocks.post).toHaveBeenCalled();
   });
 });
diff --git a/vite.config.js b/vite.config.js
index 8d6866d..50032d0 100644
--- a/vite.config.js
+++ b/vite.config.js
@@ -1,5 +1,6 @@
 import { sveltekit } from '@sveltejs/kit/vite';
 import { defineConfig } from 'vite';
+import { configDefaults } from 'vitest/config'
 
 export default defineConfig({
   plugins: [sveltekit()],
@@ -7,7 +8,21 @@ export default defineConfig({
     // If you are testing components client-side, you need to setup a DOM environment.
     // If not all your files should have this environment, you can use a
     // `// @vitest-environment jsdom` comment at the top of the test files instead.
-    environment: 'jsdom'
+    environment: 'jsdom',
+    coverage: {
+      thresholds: {
+        statements: 31,
+        branches: 46,
+        functions: 8,
+        lines: 31
+      },
+      exclude: [
+        ...configDefaults.exclude,
+        '**/*.config.js',
+        '**/docs/**',
+        '**/target/**'
+      ]
+    }
   },
   // Tell Vitest to use the `browser` entry points in `package.json` files,
   // even though it's running in Node
-- 
cgit v1.2.3


From a5945c50c41a0a41104a0c28ad1279ea3f9255f9 Mon Sep 17 00:00:00 2001
From: Kit La Touche <kit@transneptune.net>
Date: Fri, 15 Nov 2024 22:45:53 -0500
Subject: Ratchet up coverage

Just basic "will it mount?" tests for more components.
---
 .../lib/components/ActiveChannel.svelte.test.js    | 22 ++++++++++++++++++++++
 ui/tests/lib/components/Channel.svelte.test.js     | 22 ++++++++++++++++++++++
 ui/tests/lib/components/ChannelList.svelte.test.js | 22 ++++++++++++++++++++++
 ui/tests/lib/components/LogIn.svelte.test.js       | 22 ++++++++++++++++++++++
 ui/tests/lib/components/MessageRun.svelte.test.js  | 22 ++++++++++++++++++++++
 vite.config.js                                     |  8 ++++----
 6 files changed, 114 insertions(+), 4 deletions(-)
 create mode 100644 ui/tests/lib/components/ActiveChannel.svelte.test.js
 create mode 100644 ui/tests/lib/components/Channel.svelte.test.js
 create mode 100644 ui/tests/lib/components/ChannelList.svelte.test.js
 create mode 100644 ui/tests/lib/components/LogIn.svelte.test.js
 create mode 100644 ui/tests/lib/components/MessageRun.svelte.test.js

(limited to 'ui')

diff --git a/ui/tests/lib/components/ActiveChannel.svelte.test.js b/ui/tests/lib/components/ActiveChannel.svelte.test.js
new file mode 100644
index 0000000..34e92d3
--- /dev/null
+++ b/ui/tests/lib/components/ActiveChannel.svelte.test.js
@@ -0,0 +1,22 @@
+import { flushSync, mount, unmount } from 'svelte';
+import { afterEach, beforeEach, describe, expect, test } from 'vitest';
+import ActiveChannel from '$lib/components/ActiveChannel.svelte';
+
+let component;
+
+describe('ActiveChannel', () => {
+  beforeEach(() => {
+    component = mount(ActiveChannel, {
+      target: document.body, // `document` exists because of jsdom
+    });
+    flushSync();
+  });
+
+  afterEach(() => {
+    unmount(component);
+  });
+
+  test('mounts', async () => {
+    expect(component).toBeTruthy();
+  });
+});
diff --git a/ui/tests/lib/components/Channel.svelte.test.js b/ui/tests/lib/components/Channel.svelte.test.js
new file mode 100644
index 0000000..e43fa12
--- /dev/null
+++ b/ui/tests/lib/components/Channel.svelte.test.js
@@ -0,0 +1,22 @@
+import { flushSync, mount, unmount } from 'svelte';
+import { afterEach, beforeEach, describe, expect, test } from 'vitest';
+import Channel from '$lib/components/Channel.svelte';
+
+let component;
+
+describe('Channel', () => {
+  beforeEach(() => {
+    component = mount(Channel, {
+      target: document.body, // `document` exists because of jsdom
+    });
+    flushSync();
+  });
+
+  afterEach(() => {
+    unmount(component);
+  });
+
+  test('mounts', async () => {
+    expect(component).toBeTruthy();
+  });
+});
diff --git a/ui/tests/lib/components/ChannelList.svelte.test.js b/ui/tests/lib/components/ChannelList.svelte.test.js
new file mode 100644
index 0000000..e6ec64d
--- /dev/null
+++ b/ui/tests/lib/components/ChannelList.svelte.test.js
@@ -0,0 +1,22 @@
+import { flushSync, mount, unmount } from 'svelte';
+import { afterEach, beforeEach, describe, expect, test } from 'vitest';
+import ChannelList from '$lib/components/ChannelList.svelte';
+
+let component;
+
+describe('ChannelList', () => {
+  beforeEach(() => {
+    component = mount(ChannelList, {
+      target: document.body, // `document` exists because of jsdom
+    });
+    flushSync();
+  });
+
+  afterEach(() => {
+    unmount(component);
+  });
+
+  test('mounts', async () => {
+    expect(component).toBeTruthy();
+  });
+});
diff --git a/ui/tests/lib/components/LogIn.svelte.test.js b/ui/tests/lib/components/LogIn.svelte.test.js
new file mode 100644
index 0000000..3b1d299
--- /dev/null
+++ b/ui/tests/lib/components/LogIn.svelte.test.js
@@ -0,0 +1,22 @@
+import { flushSync, mount, unmount } from 'svelte';
+import { afterEach, beforeEach, describe, expect, test } from 'vitest';
+import LogIn from '$lib/components/LogIn.svelte';
+
+let component;
+
+describe('LogIn', () => {
+  beforeEach(() => {
+    component = mount(LogIn, {
+      target: document.body, // `document` exists because of jsdom
+    });
+    flushSync();
+  });
+
+  afterEach(() => {
+    unmount(component);
+  });
+
+  test('mounts', async () => {
+    expect(component).toBeTruthy();
+  });
+});
diff --git a/ui/tests/lib/components/MessageRun.svelte.test.js b/ui/tests/lib/components/MessageRun.svelte.test.js
new file mode 100644
index 0000000..480ce98
--- /dev/null
+++ b/ui/tests/lib/components/MessageRun.svelte.test.js
@@ -0,0 +1,22 @@
+import { flushSync, mount, unmount } from 'svelte';
+import { afterEach, beforeEach, describe, expect, test } from 'vitest';
+import MessageRun from '$lib/components/MessageRun.svelte';
+
+let component;
+
+describe('MessageRun', () => {
+  beforeEach(() => {
+    component = mount(MessageRun, {
+      target: document.body, // `document` exists because of jsdom
+    });
+    flushSync();
+  });
+
+  afterEach(() => {
+    unmount(component);
+  });
+
+  test('mounts', async () => {
+    expect(component).toBeTruthy();
+  });
+});
diff --git a/vite.config.js b/vite.config.js
index 50032d0..e74647a 100644
--- a/vite.config.js
+++ b/vite.config.js
@@ -11,10 +11,10 @@ export default defineConfig({
     environment: 'jsdom',
     coverage: {
       thresholds: {
-        statements: 31,
-        branches: 46,
-        functions: 8,
-        lines: 31
+        statements: 49,
+        branches: 60,
+        functions: 14,
+        lines: 49
       },
       exclude: [
         ...configDefaults.exclude,
-- 
cgit v1.2.3


From 0be2370e0d332b5b8ae2548f6ffdd7869d441ab8 Mon Sep 17 00:00:00 2001
From: Kit La Touche <kit@transneptune.net>
Date: Mon, 18 Nov 2024 22:20:08 -0500
Subject: Add Tinygesture for swipe events

Hide and show channel menu thus. It doesn't gradually pull it out, which
is less than ideal, but it's good enough for now.
---
 package-lock.json              |  9 +++++-
 package.json                   |  3 +-
 ui/routes/(app)/+layout.svelte | 64 ++++++++++++++++++++----------------------
 3 files changed, 41 insertions(+), 35 deletions(-)

(limited to 'ui')

diff --git a/package-lock.json b/package-lock.json
index 01b0cce..179731a 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -10,7 +10,8 @@
 			"dependencies": {
 				"axios": "^1.7.7",
 				"dompurify": "^3.1.7",
-				"marked": "^14.1.3"
+				"marked": "^14.1.3",
+				"tinygesture": "^3.0.0"
 			},
 			"devDependencies": {
 				"@skeletonlabs/skeleton": "^2.10.3",
@@ -3372,6 +3373,12 @@
 				"globrex": "^0.1.2"
 			}
 		},
+		"node_modules/tinygesture": {
+			"version": "3.0.0",
+			"resolved": "https://registry.npmjs.org/tinygesture/-/tinygesture-3.0.0.tgz",
+			"integrity": "sha512-UawUggtPCHy+N65ULpR/i6VLH8AzB7jWVvTNoXRFTJNh+ub6lP/SJCxzV/Ua7sJbCt9U9I79wKkKk3wbjcLdbQ==",
+			"license": "Apache-2.0"
+		},
 		"node_modules/to-regex-range": {
 			"version": "5.0.1",
 			"resolved": "https://registry.npmjs.org/to-regex-range/-/to-regex-range-5.0.1.tgz",
diff --git a/package.json b/package.json
index c51974c..0b96558 100644
--- a/package.json
+++ b/package.json
@@ -37,6 +37,7 @@
 	"dependencies": {
 		"axios": "^1.7.7",
 		"dompurify": "^3.1.7",
-		"marked": "^14.1.3"
+		"marked": "^14.1.3",
+		"tinygesture": "^3.0.0"
 	}
 }
diff --git a/ui/routes/(app)/+layout.svelte b/ui/routes/(app)/+layout.svelte
index 84c71ec..406d85a 100644
--- a/ui/routes/(app)/+layout.svelte
+++ b/ui/routes/(app)/+layout.svelte
@@ -1,7 +1,9 @@
 <script>
   import { page } from '$app/stores';
   import { goto } from '$app/navigation';
+  import { browser } from '$app/environment';
   import { onMount, onDestroy, getContext } from 'svelte';
+  import TinyGesture from 'tinygesture';
 
   import { boot, subscribeToEvents } from '$lib/apiServer';
   import { currentUser, logins, channelsList, messages } from '$lib/store';
@@ -10,6 +12,7 @@
   import CreateChannelForm from '$lib/components/CreateChannelForm.svelte';
 
   let events = null;
+  let gesture = null;
 
   let pageContext = getContext('page');
   let { children } = $props();
@@ -26,38 +29,18 @@
     messages.update((value) => value.setMessages(boot.messages));
   }
 
-  function setTouchEvents() {
-      document.addEventListener("touchstart", processTouchStart);
-      document.addEventListener("touchmove", processTouchMove);
-      document.addEventListener("touchcancel", processTouchCancel);
-      document.addEventListener("touchend", processTouchEnd);
-  }
-
-  function processTouchStart(ev) {
-      ev.preventDefault();
-      switch (ev.touches.length) {
-          case 1:
-              console.log("touch: single");
-              break;
-          case 2:
-              console.log("touch: double");
-              break;
-          case 3:
-              console.log("touch: triple");
-              break;
-          default:
-              console.log("touch: unsupported");
-              break;
-      }
-  }
-
-  function processTouchMove(ev) {
-  }
-
-  function processTouchCancel(ev) {
-  }
-
-  function processTouchEnd(ev) {
+  function setUpGestures() {
+    if (!browser) {
+      // Meaningless if we're not in a browser, so...
+      return;
+    }
+    gesture = new TinyGesture(window);
+    gesture.on('swiperight', (event) => {
+      pageContext.showMenu = true;
+    });
+    gesture.on('swipeleft', (event) => {
+      pageContext.showMenu = false;
+    });
   }
 
   onMount(async () => {
@@ -79,14 +62,18 @@
         // TODO: display error.
         break;
     }
+    setUpGestures();
+
     loading = false;
-    setTouchEvents();
   });
 
   onDestroy(async () => {
     if (events !== null) {
       events.close();
     }
+    if (gesture !== null) {
+        gesture.destroy();
+    }
   });
 </script>
 
@@ -113,12 +100,23 @@
 {/if}
 
 <style>
+  /* Just some global CSS variables, don't mind them.
+   */
   :root {
     --app-bar-height: 68px;
     --input-row-height: 2rem;
     --interface-padding: 16px;
   }
 
+  /* This should help minimize swipe-to-go-back behaviour, enabling our
+   * swipe-to-reveal-channel-menu behaviour. It won't work in all cases; in iOS
+   * Safari, when swiping from the screen edge, the OS gets th event and
+   * handles it before the browser does.
+   */
+  html, body {
+      overscroll-behavior-x: none;
+  }
+
   #interface {
     margin: unset;
     display: grid;
-- 
cgit v1.2.3


From 7b174741b5ed53ec9f1504044e044be1a7992289 Mon Sep 17 00:00:00 2001
From: Owen Jacobson <owen@grimoire.ca>
Date: Tue, 19 Nov 2024 12:28:15 -0500
Subject: npm run format

---
 ui/tests/lib/components/ActiveChannel.svelte.test.js     |  2 +-
 ui/tests/lib/components/ChangePassword.svelte.test.js    | 10 +++++-----
 ui/tests/lib/components/Channel.svelte.test.js           |  2 +-
 ui/tests/lib/components/ChannelList.svelte.test.js       |  2 +-
 ui/tests/lib/components/CreateChannelForm.svelte.test.js | 10 +++++-----
 ui/tests/lib/components/LogIn.svelte.test.js             |  2 +-
 ui/tests/lib/components/MessageInput.svelte.test.js      | 10 +++++-----
 ui/tests/lib/components/MessageRun.svelte.test.js        |  2 +-
 8 files changed, 20 insertions(+), 20 deletions(-)

(limited to 'ui')

diff --git a/ui/tests/lib/components/ActiveChannel.svelte.test.js b/ui/tests/lib/components/ActiveChannel.svelte.test.js
index 34e92d3..183c823 100644
--- a/ui/tests/lib/components/ActiveChannel.svelte.test.js
+++ b/ui/tests/lib/components/ActiveChannel.svelte.test.js
@@ -7,7 +7,7 @@ let component;
 describe('ActiveChannel', () => {
   beforeEach(() => {
     component = mount(ActiveChannel, {
-      target: document.body, // `document` exists because of jsdom
+      target: document.body // `document` exists because of jsdom
     });
     flushSync();
   });
diff --git a/ui/tests/lib/components/ChangePassword.svelte.test.js b/ui/tests/lib/components/ChangePassword.svelte.test.js
index 5773eae..9eb40f5 100644
--- a/ui/tests/lib/components/ChangePassword.svelte.test.js
+++ b/ui/tests/lib/components/ChangePassword.svelte.test.js
@@ -6,7 +6,7 @@ import axios from 'axios';
 let component;
 
 let mocks = vi.hoisted(() => ({
-  post: vi.fn(),
+  post: vi.fn()
 }));
 
 describe('ChangePassword', () => {
@@ -19,9 +19,9 @@ describe('ChangePassword', () => {
           ...actual.default,
           create: vi.fn(() => ({
             ...actual.default.create(),
-            post: mocks.post,
-          })),
-        },
+            post: mocks.post
+          }))
+        }
       };
 
       return mockAxios;
@@ -29,7 +29,7 @@ describe('ChangePassword', () => {
     mocks.post.mockResolvedValue({ status: 200 });
 
     component = mount(ChangePassword, {
-      target: document.body, // `document` exists because of jsdom
+      target: document.body // `document` exists because of jsdom
     });
     flushSync();
   });
diff --git a/ui/tests/lib/components/Channel.svelte.test.js b/ui/tests/lib/components/Channel.svelte.test.js
index e43fa12..a6fdab9 100644
--- a/ui/tests/lib/components/Channel.svelte.test.js
+++ b/ui/tests/lib/components/Channel.svelte.test.js
@@ -7,7 +7,7 @@ let component;
 describe('Channel', () => {
   beforeEach(() => {
     component = mount(Channel, {
-      target: document.body, // `document` exists because of jsdom
+      target: document.body // `document` exists because of jsdom
     });
     flushSync();
   });
diff --git a/ui/tests/lib/components/ChannelList.svelte.test.js b/ui/tests/lib/components/ChannelList.svelte.test.js
index e6ec64d..7b5ae4a 100644
--- a/ui/tests/lib/components/ChannelList.svelte.test.js
+++ b/ui/tests/lib/components/ChannelList.svelte.test.js
@@ -7,7 +7,7 @@ let component;
 describe('ChannelList', () => {
   beforeEach(() => {
     component = mount(ChannelList, {
-      target: document.body, // `document` exists because of jsdom
+      target: document.body // `document` exists because of jsdom
     });
     flushSync();
   });
diff --git a/ui/tests/lib/components/CreateChannelForm.svelte.test.js b/ui/tests/lib/components/CreateChannelForm.svelte.test.js
index c1adba5..a4a4695 100644
--- a/ui/tests/lib/components/CreateChannelForm.svelte.test.js
+++ b/ui/tests/lib/components/CreateChannelForm.svelte.test.js
@@ -6,7 +6,7 @@ import axios from 'axios';
 let component;
 
 let mocks = vi.hoisted(() => ({
-  post: vi.fn(),
+  post: vi.fn()
 }));
 
 describe('CreateChannelForm', () => {
@@ -19,9 +19,9 @@ describe('CreateChannelForm', () => {
           ...actual.default,
           create: vi.fn(() => ({
             ...actual.default.create(),
-            post: mocks.post,
-          })),
-        },
+            post: mocks.post
+          }))
+        }
       };
 
       return mockAxios;
@@ -29,7 +29,7 @@ describe('CreateChannelForm', () => {
     mocks.post.mockResolvedValue({ status: 200 });
 
     component = mount(CreateChannelForm, {
-      target: document.body, // `document` exists because of jsdom
+      target: document.body // `document` exists because of jsdom
     });
     flushSync();
   });
diff --git a/ui/tests/lib/components/LogIn.svelte.test.js b/ui/tests/lib/components/LogIn.svelte.test.js
index 3b1d299..b64d846 100644
--- a/ui/tests/lib/components/LogIn.svelte.test.js
+++ b/ui/tests/lib/components/LogIn.svelte.test.js
@@ -7,7 +7,7 @@ let component;
 describe('LogIn', () => {
   beforeEach(() => {
     component = mount(LogIn, {
-      target: document.body, // `document` exists because of jsdom
+      target: document.body // `document` exists because of jsdom
     });
     flushSync();
   });
diff --git a/ui/tests/lib/components/MessageInput.svelte.test.js b/ui/tests/lib/components/MessageInput.svelte.test.js
index d90026c..3dde5d7 100644
--- a/ui/tests/lib/components/MessageInput.svelte.test.js
+++ b/ui/tests/lib/components/MessageInput.svelte.test.js
@@ -6,7 +6,7 @@ import axios from 'axios';
 let component;
 
 let mocks = vi.hoisted(() => ({
-  post: vi.fn(),
+  post: vi.fn()
 }));
 
 describe('MessageInput', () => {
@@ -19,9 +19,9 @@ describe('MessageInput', () => {
           ...actual.default,
           create: vi.fn(() => ({
             ...actual.default.create(),
-            post: mocks.post,
-          })),
-        },
+            post: mocks.post
+          }))
+        }
       };
 
       return mockAxios;
@@ -29,7 +29,7 @@ describe('MessageInput', () => {
     mocks.post.mockResolvedValue({ status: 200 });
 
     component = mount(MessageInput, {
-      target: document.body, // `document` exists because of jsdom
+      target: document.body // `document` exists because of jsdom
     });
     flushSync();
   });
diff --git a/ui/tests/lib/components/MessageRun.svelte.test.js b/ui/tests/lib/components/MessageRun.svelte.test.js
index 480ce98..7671c52 100644
--- a/ui/tests/lib/components/MessageRun.svelte.test.js
+++ b/ui/tests/lib/components/MessageRun.svelte.test.js
@@ -7,7 +7,7 @@ let component;
 describe('MessageRun', () => {
   beforeEach(() => {
     component = mount(MessageRun, {
-      target: document.body, // `document` exists because of jsdom
+      target: document.body // `document` exists because of jsdom
     });
     flushSync();
   });
-- 
cgit v1.2.3


From 9a2efe0545e1e695d45f8c7577c3899a6a648e53 Mon Sep 17 00:00:00 2001
From: Owen Jacobson <owen@grimoire.ca>
Date: Tue, 19 Nov 2024 12:41:50 -0500
Subject: Patch out --host again; npm run format

---
 tools/run                      | 2 +-
 ui/routes/(app)/+layout.svelte | 7 ++++---
 2 files changed, 5 insertions(+), 4 deletions(-)

(limited to 'ui')

diff --git a/tools/run b/tools/run
index 8c450cf..7f43a19 100755
--- a/tools/run
+++ b/tools/run
@@ -5,7 +5,7 @@
 if [ -z ${PILCROW_DEV+x} ]; then
     cargo run -- "$@"
 else
-    npm run dev -- --host 192.168.68.57 & PIDS[0]=$!
+    npm run dev -- & PIDS[0]=$!
     cargo run -- "$@" & PIDS[1]=$!
 
     trap "kill ${PIDS[*]}" SIGINT
diff --git a/ui/routes/(app)/+layout.svelte b/ui/routes/(app)/+layout.svelte
index 406d85a..cc900a6 100644
--- a/ui/routes/(app)/+layout.svelte
+++ b/ui/routes/(app)/+layout.svelte
@@ -72,7 +72,7 @@
       events.close();
     }
     if (gesture !== null) {
-        gesture.destroy();
+      gesture.destroy();
     }
   });
 </script>
@@ -113,8 +113,9 @@
    * Safari, when swiping from the screen edge, the OS gets th event and
    * handles it before the browser does.
    */
-  html, body {
-      overscroll-behavior-x: none;
+  html,
+  body {
+    overscroll-behavior-x: none;
   }
 
   #interface {
-- 
cgit v1.2.3


From cbba25f939a12026816c1b8202d821f57606f248 Mon Sep 17 00:00:00 2001
From: Kit La Touche <kit@transneptune.net>
Date: Thu, 21 Nov 2024 16:25:04 -0500
Subject: Clean up extraneous variables

Just use state, don't derive from a non-state local variable.
---
 ui/lib/components/CreateChannelForm.svelte | 7 +++----
 ui/lib/components/MessageInput.svelte      | 8 +++-----
 2 files changed, 6 insertions(+), 9 deletions(-)

(limited to 'ui')

diff --git a/ui/lib/components/CreateChannelForm.svelte b/ui/lib/components/CreateChannelForm.svelte
index afa3f78..d50e60d 100644
--- a/ui/lib/components/CreateChannelForm.svelte
+++ b/ui/lib/components/CreateChannelForm.svelte
@@ -2,17 +2,16 @@
   import { createChannel } from '$lib/apiServer';
 
   let name = $state('');
-  let pending = false;
-  let disabled = $derived(pending);
+  let disabled = $state(false);
 
   async function handleSubmit(event) {
     event.preventDefault();
-    pending = true;
+    disabled = true;
     const response = await createChannel(name);
     if (200 <= response.status && response.status < 300) {
       name = '';
     }
-    pending = false;
+    disabled = false;
   }
 </script>
 
diff --git a/ui/lib/components/MessageInput.svelte b/ui/lib/components/MessageInput.svelte
index 26521e1..22456f3 100644
--- a/ui/lib/components/MessageInput.svelte
+++ b/ui/lib/components/MessageInput.svelte
@@ -5,16 +5,14 @@
 
   let form;
   let value = $state('');
-  let pending = false;
-
-  let disabled = $derived(pending);
+  let disabled = $state(false);
 
   async function onSubmit(event) {
     event.preventDefault();
-    pending = true;
+    disabled = true;
     await postToChannel(channel, value);
     form.reset();
-    pending = false;
+    disabled = false;
   }
 
   function onKeyDown(event) {
-- 
cgit v1.2.3


From c2f711c56a3bc736cf507a920dee0915b69af77e Mon Sep 17 00:00:00 2001
From: Kit La Touche <kit@transneptune.net>
Date: Thu, 21 Nov 2024 16:26:10 -0500
Subject: Use store.set where appropriate

If you're setting it to a static value, use set.
---
 ui/lib/components/LogOut.svelte | 2 +-
 ui/routes/(app)/+layout.svelte  | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

(limited to 'ui')

diff --git a/ui/lib/components/LogOut.svelte b/ui/lib/components/LogOut.svelte
index 25dd5e9..52aa039 100644
--- a/ui/lib/components/LogOut.svelte
+++ b/ui/lib/components/LogOut.svelte
@@ -7,7 +7,7 @@
     event.preventDefault();
     const response = await logOut();
     if (200 <= response.status && response.status < 300) {
-      currentUser.update(() => null);
+      currentUser.set(null);
       goto('/login');
     }
   }
diff --git a/ui/routes/(app)/+layout.svelte b/ui/routes/(app)/+layout.svelte
index cc900a6..e542f24 100644
--- a/ui/routes/(app)/+layout.svelte
+++ b/ui/routes/(app)/+layout.svelte
@@ -20,10 +20,10 @@
   let channel = $derived($page.params.channel);
 
   function onBooted(boot) {
-    currentUser.update(() => ({
+    currentUser.set({
       id: boot.login.id,
       username: boot.login.name
-    }));
+    });
     logins.update((value) => value.setLogins(boot.logins));
     channelsList.update((value) => value.setChannels(boot.channels));
     messages.update((value) => value.setMessages(boot.messages));
@@ -51,11 +51,11 @@
         events = subscribeToEvents(response.data.resume_point);
         break;
       case 401:
-        currentUser.update(() => null);
+        currentUser.set(null);
         goto('/login');
         break;
       case 503:
-        currentUser.update(() => null);
+        currentUser.set(null);
         goto('/setup');
         break;
       default:
-- 
cgit v1.2.3


From 5b2db6bbe7749d1b13a06c89a368b3b0849bc368 Mon Sep 17 00:00:00 2001
From: Owen Jacobson <owen@grimoire.ca>
Date: Tue, 26 Nov 2024 01:57:26 -0500
Subject: HTML-level styling inside non-top-level elements is not actually
 applied

---
 ui/app.css                     | 10 ++++++++++
 ui/routes/(app)/+layout.svelte | 10 ----------
 2 files changed, 10 insertions(+), 10 deletions(-)

(limited to 'ui')

diff --git a/ui/app.css b/ui/app.css
index b5c61c9..b316dea 100644
--- a/ui/app.css
+++ b/ui/app.css
@@ -1,3 +1,13 @@
 @tailwind base;
 @tailwind components;
 @tailwind utilities;
+
+/* This should help minimize swipe-to-go-back behaviour, enabling our
+* swipe-to-reveal-channel-menu behaviour. It won't work in all cases; in iOS
+* Safari, when swiping from the screen edge, the OS gets th event and
+* handles it before the browser does.
+*/
+html,
+body {
+  overscroll-behavior-x: none;
+}
diff --git a/ui/routes/(app)/+layout.svelte b/ui/routes/(app)/+layout.svelte
index e542f24..bafd6b8 100644
--- a/ui/routes/(app)/+layout.svelte
+++ b/ui/routes/(app)/+layout.svelte
@@ -108,16 +108,6 @@
     --interface-padding: 16px;
   }
 
-  /* This should help minimize swipe-to-go-back behaviour, enabling our
-   * swipe-to-reveal-channel-menu behaviour. It won't work in all cases; in iOS
-   * Safari, when swiping from the screen edge, the OS gets th event and
-   * handles it before the browser does.
-   */
-  html,
-  body {
-    overscroll-behavior-x: none;
-  }
-
   #interface {
     margin: unset;
     display: grid;
-- 
cgit v1.2.3


From 26fb3e0b364d7201cf0dfd9e524473ffa20dfa33 Mon Sep 17 00:00:00 2001
From: Owen Jacobson <owen@grimoire.ca>
Date: Tue, 26 Nov 2024 02:04:40 -0500
Subject: Best-guess an ARIA role for this element.

Svelte's lint complains if you add interaction to a div but don't give that div an ARIA role.
---
 ui/lib/components/Message.svelte | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'ui')

diff --git a/ui/lib/components/Message.svelte b/ui/lib/components/Message.svelte
index fddeecd..1663696 100644
--- a/ui/lib/components/Message.svelte
+++ b/ui/lib/components/Message.svelte
@@ -23,7 +23,7 @@
   }
 </script>
 
-<div class="message relative" class:bg-warning-800={deleteArmed} {onmouseleave}>
+<div class="message relative" class:bg-warning-800={deleteArmed} {onmouseleave} role="article">
   <div class="handle chip bg-surface-700 absolute -top-6 right-0">
     {at}
     {#if editable}
-- 
cgit v1.2.3


From 9da480d92c6d81139e77eb6542986cf588ae3386 Mon Sep 17 00:00:00 2001
From: Owen Jacobson <owen@grimoire.ca>
Date: Tue, 26 Nov 2024 02:13:39 -0500
Subject: Don't sink the top bar so far down into the page.

---
 ui/routes/(app)/+layout.svelte | 2 +-
 ui/routes/+layout.svelte       | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

(limited to 'ui')

diff --git a/ui/routes/(app)/+layout.svelte b/ui/routes/(app)/+layout.svelte
index bafd6b8..ad8227e 100644
--- a/ui/routes/(app)/+layout.svelte
+++ b/ui/routes/(app)/+layout.svelte
@@ -103,7 +103,7 @@
   /* Just some global CSS variables, don't mind them.
    */
   :root {
-    --app-bar-height: 68px;
+    --app-bar-height: 48px;
     --input-row-height: 2rem;
     --interface-padding: 16px;
   }
diff --git a/ui/routes/+layout.svelte b/ui/routes/+layout.svelte
index 8940659..26033e0 100644
--- a/ui/routes/+layout.svelte
+++ b/ui/routes/+layout.svelte
@@ -24,7 +24,7 @@
   let { children } = $props();
 </script>
 
-<AppBar class="app-bar">
+<AppBar padding="px-4 pt-0 pb-4">
   <svelte:fragment slot="lead">
     <button onclick={toggleMenu} class="cursor-pointer">
       <img class="w-8 h-8" alt="logo" src={logo} />
-- 
cgit v1.2.3