summaryrefslogtreecommitdiffstats
path: root/src/components/molecules/forms/prism-theme-toggle.test.tsx
diff options
context:
space:
mode:
authorArmand Philippot <git@armandphilippot.com>2022-06-01 22:37:56 +0200
committerGitHub <noreply@github.com>2022-06-01 22:37:56 +0200
commit0a33a4658d848fe056715c6da053763407845b2a (patch)
tree7c679e54ba4bbadaf0a59bbde780f5742e3b875d /src/components/molecules/forms/prism-theme-toggle.test.tsx
parent97031a86ca38890e60ecec79828498b7bb13cbfa (diff)
parent6be20422494e3806fba3d1c5ad5c3e98bd6e67e5 (diff)
chore(a11y): improve website settings accessibility (#17)
The previous switch buttons (using checkbox) was not a11y compliant. So I change my approach to use radio buttons and to clearly separate the two different states. I also convert the Ackee select setting to improve consistency between settings.
Diffstat (limited to 'src/components/molecules/forms/prism-theme-toggle.test.tsx')
-rw-r--r--src/components/molecules/forms/prism-theme-toggle.test.tsx4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/components/molecules/forms/prism-theme-toggle.test.tsx b/src/components/molecules/forms/prism-theme-toggle.test.tsx
index c9d7894..91e8e2e 100644
--- a/src/components/molecules/forms/prism-theme-toggle.test.tsx
+++ b/src/components/molecules/forms/prism-theme-toggle.test.tsx
@@ -5,8 +5,8 @@ describe('PrismThemeToggle', () => {
it('renders a toggle component', () => {
render(<PrismThemeToggle />);
expect(
- screen.getByRole('checkbox', {
- name: `Code blocks: Light theme Dark theme`,
+ screen.getByRole('radiogroup', {
+ name: /Code blocks:/i,
})
).toBeInTheDocument();
});