From 07e5f260ad4c88879f27c3fc8742155f872e8994 Mon Sep 17 00:00:00 2001
From: Owen Jacobson <owen@grimoire.ca>
Date: Thu, 22 May 2025 00:21:35 -0400
Subject: Use browser-native methods to hide elements, not CSS alone.

The hidden `textarea` used to attach the form value to the DOM was being included in the ARIA accessibility tree, at least in testing (I didn't check in a browser). While we could suppress this iwth `aria-role="hidden"`, the WHATWG recommendation is to Not Do That, and to find another way to hide the element, instead. Marking the element as hidden accomplishes that goal, _and_ gets rid of a style rule.
---
 ui/lib/components/MessageInput.svelte | 2 +-
 ui/styles/forms.css                   | 4 ----
 2 files changed, 1 insertion(+), 5 deletions(-)

(limited to 'ui')

diff --git a/ui/lib/components/MessageInput.svelte b/ui/lib/components/MessageInput.svelte
index cdb855e..30038c6 100644
--- a/ui/lib/components/MessageInput.svelte
+++ b/ui/lib/components/MessageInput.svelte
@@ -46,7 +46,7 @@
 </script>
 
 <form {onsubmit}>
-  <textarea bind:value class="hidden" {disabled}></textarea>
+  <textarea bind:value hidden {disabled}></textarea>
   <div
     role="textbox"
     tabindex="0"
diff --git a/ui/styles/forms.css b/ui/styles/forms.css
index 98fd43e..f4d218f 100644
--- a/ui/styles/forms.css
+++ b/ui/styles/forms.css
@@ -26,7 +26,3 @@ form.form > button {
 .disabled {
   color: var(--light-text);
 }
-
-.hidden {
-  display: none;
-}
-- 
cgit v1.2.3