From f01420258f2eea281e812a917ef0fb38480d0b8c Mon Sep 17 00:00:00 2001
From: Arnold Loubriat <datatriny@gmail.com>
Date: Sun, 15 Sep 2024 18:56:27 +0200
Subject: [PATCH] Changelog: Introduce `AccessibleRole::ListItem`

---
 internal/backends/qt/qt_accessible.rs         | 1 +
 internal/backends/testing/slint_systest.proto | 1 +
 internal/backends/testing/systest.rs          | 2 ++
 internal/backends/winit/accesskit.rs          | 1 +
 internal/common/enums.rs                      | 2 ++
 5 files changed, 7 insertions(+)

diff --git a/internal/backends/qt/qt_accessible.rs b/internal/backends/qt/qt_accessible.rs
index 4ad25ea09..11be06273 100644
--- a/internal/backends/qt/qt_accessible.rs
+++ b/internal/backends/qt/qt_accessible.rs
@@ -320,6 +320,7 @@ cpp! {{
                     i_slint_core::items::AccessibleRole::Tree => QAccessible_Role_Tree,
                     i_slint_core::items::AccessibleRole::TextInput => QAccessible_Role_EditableText,
                     i_slint_core::items::AccessibleRole::Switch => QAccessible_Role_CheckBox,
+                    i_slint_core::items::AccessibleRole::ListItem => QAccessible_Role_ListItem,
                     _ => QAccessible_Role_NoRole,
                 }
             });
diff --git a/internal/backends/testing/slint_systest.proto b/internal/backends/testing/slint_systest.proto
index 4dc0923db..0aad5ecaa 100644
--- a/internal/backends/testing/slint_systest.proto
+++ b/internal/backends/testing/slint_systest.proto
@@ -96,6 +96,7 @@ enum AccessibleRole {
     ProgressIndicator = 12;
     TextInput = 13;
     Switch = 14;
+    ListItem = 15;
 }
 
 message ElementQueryInstruction {
diff --git a/internal/backends/testing/systest.rs b/internal/backends/testing/systest.rs
index 5796dc4dc..ecbfc9ec2 100644
--- a/internal/backends/testing/systest.rs
+++ b/internal/backends/testing/systest.rs
@@ -526,6 +526,7 @@ fn convert_to_proto_accessible_role(
         }
         i_slint_core::items::AccessibleRole::TextInput => proto::AccessibleRole::TextInput,
         i_slint_core::items::AccessibleRole::Switch => proto::AccessibleRole::Switch,
+        i_slint_core::items::AccessibleRole::ListItem => proto::AccessibleRole::ListItem,
         _ => return None,
     })
 }
@@ -551,6 +552,7 @@ fn convert_from_proto_accessible_role(
         }
         proto::AccessibleRole::TextInput => i_slint_core::items::AccessibleRole::TextInput,
         proto::AccessibleRole::Switch => i_slint_core::items::AccessibleRole::Switch,
+        proto::AccessibleRole::ListItem => i_slint_core::items::AccessibleRole::ListItem,
     })
 }
 
diff --git a/internal/backends/winit/accesskit.rs b/internal/backends/winit/accesskit.rs
index 8279166be..ee02a7db4 100644
--- a/internal/backends/winit/accesskit.rs
+++ b/internal/backends/winit/accesskit.rs
@@ -368,6 +368,7 @@ impl NodeCollection {
                         Role::ProgressIndicator
                     }
                     i_slint_core::items::AccessibleRole::Switch => Role::Switch,
+                    i_slint_core::items::AccessibleRole::ListItem => Role::ListBoxOption,
                     _ => Role::Unknown,
                 },
                 item.accessible_string_property(
diff --git a/internal/common/enums.rs b/internal/common/enums.rs
index 50f4df89a..6dfb0674d 100644
--- a/internal/common/enums.rs
+++ b/internal/common/enums.rs
@@ -369,6 +369,8 @@ macro_rules! for_each_enums {
                 TextInput,
                 /// The element is a [`Switch`](slint-reference:src/language/widgets/switch) or behaves like one.
                 Switch,
+                /// The element is an item in a [`ListView`](slint-reference:src/language/widgets/listview).
+                ListItem,
             }
 
             /// This enum represents the different values of the `sort-order` property.