From 01368abc9893caa7acd3d632fd7a2d4284e64167 Mon Sep 17 00:00:00 2001
From: Owen Jacobson <owen@grimoire.ca>
Date: Fri, 21 Feb 2025 16:21:43 -0500
Subject: Remove unused type attribute on textarea

---
 ui/lib/components/MessageInput.svelte | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

(limited to 'ui/lib/components')

diff --git a/ui/lib/components/MessageInput.svelte b/ui/lib/components/MessageInput.svelte
index 5869654..1eb1d7b 100644
--- a/ui/lib/components/MessageInput.svelte
+++ b/ui/lib/components/MessageInput.svelte
@@ -24,7 +24,6 @@
 </script>
 
 <form bind:this={form} onsubmit={onSubmit}>
-  <textarea onkeydown={onKeyDown} bind:value {disabled} type="search" placeholder="Say something..."
-  ></textarea>
+  <textarea onkeydown={onKeyDown} bind:value {disabled} placeholder="Say something..."></textarea>
   <button type="submit">&raquo;</button>
 </form>
-- 
cgit v1.2.3


From 5245aba8b80f458d25eb3249c0b8cc3fe744311d Mon Sep 17 00:00:00 2001
From: Owen Jacobson <owen@grimoire.ca>
Date: Fri, 21 Feb 2025 16:27:00 -0500
Subject: Be a bit more careful with the nesting of anchors and list items.

Browsers cope with weird nestings mostly fine, but there's no upside for us in testing that.
---
 ui/lib/components/Channel.svelte | 8 ++++----
 ui/styles/sidebar.css            | 3 ++-
 2 files changed, 6 insertions(+), 5 deletions(-)

(limited to 'ui/lib/components')

diff --git a/ui/lib/components/Channel.svelte b/ui/lib/components/Channel.svelte
index c73340f..4f908d2 100644
--- a/ui/lib/components/Channel.svelte
+++ b/ui/lib/components/Channel.svelte
@@ -2,13 +2,13 @@
   let { id, name, active, hasUnreads } = $props();
 </script>
 
-<a href="/ch/{id}">
-  <li class:active>
+<li class:active>
+  <a href="/ch/{id}">
     {#if hasUnreads}
       <span class="badge has-unreads">❦</span>
     {:else}
       <span class="badge has-no-unreads">¶</span>
     {/if}
     <span>{name}</span>
-  </li>
-</a>
+  </a>
+</li>
diff --git a/ui/styles/sidebar.css b/ui/styles/sidebar.css
index 5e5e16a..c6aab6a 100644
--- a/ui/styles/sidebar.css
+++ b/ui/styles/sidebar.css
@@ -4,6 +4,8 @@
 }
 
 .list-nav a {
+  display: block;
+  padding: 0.5rem;
   text-decoration: none;
 }
 
@@ -12,7 +14,6 @@
 }
 
 .list-nav li {
-  padding: 0.5rem;
   border-radius: 0.5rem;
   border: 1px solid var(--colour-navbar-border);
   margin: 0.25rem;
-- 
cgit v1.2.3


From b9bc643af0d0523a70d76236d9277539da083b36 Mon Sep 17 00:00:00 2001
From: Owen Jacobson <owen@grimoire.ca>
Date: Fri, 21 Feb 2025 16:35:56 -0500
Subject: Add missing awaits on goto() calls

---
 ui/lib/components/LogOut.svelte                | 2 +-
 ui/routes/(app)/+layout.svelte                 | 8 ++++----
 ui/routes/(login)/invite/[invite]/+page.svelte | 2 +-
 ui/routes/(login)/login/+page.svelte           | 2 +-
 ui/routes/(login)/setup/+page.svelte           | 2 +-
 5 files changed, 8 insertions(+), 8 deletions(-)

(limited to 'ui/lib/components')

diff --git a/ui/lib/components/LogOut.svelte b/ui/lib/components/LogOut.svelte
index b699cfd..1cb8fb5 100644
--- a/ui/lib/components/LogOut.svelte
+++ b/ui/lib/components/LogOut.svelte
@@ -8,7 +8,7 @@
     const response = await logOut();
     if (200 <= response.status && response.status < 300) {
       currentUser.set(null);
-      goto('/login');
+      await goto('/login');
     }
   }
 </script>
diff --git a/ui/routes/(app)/+layout.svelte b/ui/routes/(app)/+layout.svelte
index dff9245..a81efce 100644
--- a/ui/routes/(app)/+layout.svelte
+++ b/ui/routes/(app)/+layout.svelte
@@ -2,11 +2,11 @@
   import { page } from '$app/stores';
   import { goto } from '$app/navigation';
   import { browser } from '$app/environment';
-  import { onMount, onDestroy, getContext } from 'svelte';
+  import { getContext, onDestroy, onMount } from 'svelte';
   import TinyGesture from 'tinygesture';
 
   import { boot, subscribeToEvents } from '$lib/apiServer';
-  import { currentUser, logins, channelsList, messages, onEvent } from '$lib/store';
+  import { channelsList, currentUser, logins, messages, onEvent } from '$lib/store';
 
   import ChannelList from '$lib/components/ChannelList.svelte';
   import CreateChannelForm from '$lib/components/CreateChannelForm.svelte';
@@ -76,11 +76,11 @@
         break;
       case 401:
         currentUser.set(null);
-        goto('/login');
+        await goto('/login');
         break;
       case 503:
         currentUser.set(null);
-        goto('/setup');
+        await goto('/setup');
         break;
       default:
         // TODO: display error.
diff --git a/ui/routes/(login)/invite/[invite]/+page.svelte b/ui/routes/(login)/invite/[invite]/+page.svelte
index 132cbc1..0c01286 100644
--- a/ui/routes/(login)/invite/[invite]/+page.svelte
+++ b/ui/routes/(login)/invite/[invite]/+page.svelte
@@ -18,7 +18,7 @@
     if (200 <= response.status && response.status < 300) {
       username = '';
       password = '';
-      goto('/');
+      await goto('/');
     }
     pending = false;
   }
diff --git a/ui/routes/(login)/login/+page.svelte b/ui/routes/(login)/login/+page.svelte
index a1291ea..9157cef 100644
--- a/ui/routes/(login)/login/+page.svelte
+++ b/ui/routes/(login)/login/+page.svelte
@@ -16,7 +16,7 @@
     if (200 <= response.status && response.status < 300) {
       username = '';
       password = '';
-      goto('/');
+      await goto('/');
     }
     pending = false;
   }
diff --git a/ui/routes/(login)/setup/+page.svelte b/ui/routes/(login)/setup/+page.svelte
index f162ded..c63f198 100644
--- a/ui/routes/(login)/setup/+page.svelte
+++ b/ui/routes/(login)/setup/+page.svelte
@@ -16,7 +16,7 @@
     if (200 <= response.status && response.status < 300) {
       username = '';
       password = '';
-      goto('/');
+      await goto('/');
     }
     pending = false;
   }
-- 
cgit v1.2.3