diff options
| author | Owen Jacobson <owen@grimoire.ca> | 2025-05-22 00:21:35 -0400 |
|---|---|---|
| committer | Owen Jacobson <owen@grimoire.ca> | 2025-05-22 00:21:35 -0400 |
| commit | 07e5f260ad4c88879f27c3fc8742155f872e8994 (patch) | |
| tree | 08688b851ee2d2687bebf5516bfc56074afa2f7c /ui/lib/components | |
| parent | b3fcd627ae57cd0587363eaf2d5f4635a82bda60 (diff) | |
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.
Diffstat (limited to 'ui/lib/components')
| -rw-r--r-- | ui/lib/components/MessageInput.svelte | 2 |
1 files changed, 1 insertions, 1 deletions
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" |