Skip to content

Commit

Permalink
Fix accessibility issues with form
Browse files Browse the repository at this point in the history
  • Loading branch information
dar5hak committed Nov 22, 2024
1 parent a3f586a commit cf86d95
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions src/App.svelte
Original file line number Diff line number Diff line change
Expand Up @@ -159,17 +159,15 @@
oninput={handleDerive}
/>

<button class="info" tabindex="-1">
<button aria-label="Info" class="info" tabindex="-1">
<i
class="ri-information-line"
title="Site is not case-sensitive. “GitHub” equals “github”."
aria-label="Site is not case-sensitive. “GitHub” equals “github”."
></i>
</button>

<label for="cipher">Cipher key:</label>

<!-- Svelte won't allow bind:value when type is dynamic. -->
<input
type={showCipher ? "text" : "password"}
id="cipher"
Expand All @@ -182,14 +180,14 @@
/>

<button
aria-label="Show/hide cipher key"
class="toggle"
disabled={!cipher}
onclick={() => (showCipher = !showCipher)}
>
<i
class={showCipher ? "ri-eye-line" : "ri-eye-off-line"}
title="Show/hide cipher key"
aria-label="Show/hide cipher key"
></i>
</button>

Expand All @@ -204,14 +202,14 @@
/>

<button
aria-label="Show/hide password"
class="toggle"
disabled={!derivedPassword}
onclick={() => (showDerivedPassword = !showDerivedPassword)}
>
<i
class={showDerivedPassword ? "ri-eye-line" : "ri-eye-off-line"}
title="Show/hide password"
aria-label="Show/hide password"
></i>
</button>

Expand Down

0 comments on commit cf86d95

Please sign in to comment.