From fdf9d993abf6d70a127f94bb9b06ce38f0ff9c90 Mon Sep 17 00:00:00 2001
From: Jonathan Carle <jonathan_leo.carle@mail.schwarz>
Date: Mon, 25 Mar 2024 14:00:02 +0100
Subject: [PATCH 1/2] add switch role to OnyxSwitch

---
 packages/sit-onyx/src/components/OnyxSwitch/OnyxSwitch.vue | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/packages/sit-onyx/src/components/OnyxSwitch/OnyxSwitch.vue b/packages/sit-onyx/src/components/OnyxSwitch/OnyxSwitch.vue
index 547f2be7f..b88b78603 100644
--- a/packages/sit-onyx/src/components/OnyxSwitch/OnyxSwitch.vue
+++ b/packages/sit-onyx/src/components/OnyxSwitch/OnyxSwitch.vue
@@ -73,11 +73,15 @@ watch(
     :class="[requiredTypeClass]"
     :title="props.hideLabel ? props.label : undefined"
   >
+    <!-- Linter incorrectly finds an error. For a native `input` the `aria-checked` is not necessary. There is an open issue about it: https://github.com/vue-a11y/eslint-plugin-vuejs-accessibility/issues/932  -->
+    <!-- eslint-disable vuejs-accessibility/role-has-required-aria-props -->
+    <!-- TODO: disable can be merged when https://github.com/vue-a11y/eslint-plugin-vuejs-accessibility/pull/1071 was released -->
     <input
       ref="inputElement"
       v-model="isChecked"
-      :class="{ 'onyx-switch__input': true, 'onyx-switch__loading': props.loading }"
       type="checkbox"
+      role="switch"
+      :class="{ 'onyx-switch__input': true, 'onyx-switch__loading': props.loading }"
       :aria-label="props.hideLabel ? props.label : undefined"
       :disabled="props.disabled || props.loading"
       :required="props.required"

From 7974fb708cc76cd020582ee13e52cf0dffb4823d Mon Sep 17 00:00:00 2001
From: Jonathan <jonathan_leo.carle@mail.schwarz>
Date: Mon, 25 Mar 2024 15:48:18 +0100
Subject: [PATCH 2/2] Update
 packages/sit-onyx/src/components/OnyxSwitch/OnyxSwitch.vue

Co-authored-by: Linda Bopp <151019977+BoppLi@users.noreply.github.com>
---
 packages/sit-onyx/src/components/OnyxSwitch/OnyxSwitch.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/sit-onyx/src/components/OnyxSwitch/OnyxSwitch.vue b/packages/sit-onyx/src/components/OnyxSwitch/OnyxSwitch.vue
index b88b78603..72e1796b4 100644
--- a/packages/sit-onyx/src/components/OnyxSwitch/OnyxSwitch.vue
+++ b/packages/sit-onyx/src/components/OnyxSwitch/OnyxSwitch.vue
@@ -75,7 +75,7 @@ watch(
   >
     <!-- Linter incorrectly finds an error. For a native `input` the `aria-checked` is not necessary. There is an open issue about it: https://github.com/vue-a11y/eslint-plugin-vuejs-accessibility/issues/932  -->
     <!-- eslint-disable vuejs-accessibility/role-has-required-aria-props -->
-    <!-- TODO: disable can be merged when https://github.com/vue-a11y/eslint-plugin-vuejs-accessibility/pull/1071 was released -->
+    <!-- TODO: disable can be removed when https://github.com/vue-a11y/eslint-plugin-vuejs-accessibility/pull/1071 was released -->
     <input
       ref="inputElement"
       v-model="isChecked"