Commit 34544d34 authored by Nate Graham's avatar Nate Graham
Browse files

Remove unneeded accessible properties and re-arrange others

BasicListItem already sets Accessible.role: ListItem, so we don't need
to manually set that.

Set Accessible.name in the list item itself, not in its implementations.
parent c701878f
......@@ -137,8 +137,6 @@ Kirigami.ScrollablePage {
delegate: CategoryItem {
id: delegate
Accessible.role: Accessible.ListItem
Accessible.name: model.display
supportsMouseEvents: !model.IsCategoryRole || !mainColumn.searchMode
enabled: model.IsKCMRole || !mainColumn.searchMode
onClicked: {
......
......@@ -19,8 +19,9 @@ Kirigami.BasicListItem {
width: (model.DepthRole > 1 && delegate.isSearching) ? (( model.DepthRole - 1 ) * Kirigami.Units.iconSizes.smallMedium) + Kirigami.Units.largeSpacing : 0
}
text: model.display
icon: model.decoration
text: model.display
Accessible.name: model.display
trailing: RowLayout {
Rectangle {
......
......@@ -140,7 +140,6 @@ Kirigami.ScrollablePage {
delegate: CategoryItem {
id: delegate
// leftPadding: Kirigami.Units.smallSpacing + (Kirigami.Units.smallSpacing + Kirigami.Units.iconSizes.smallMedium) * (model.DepthRole - 2)
onClicked: {
systemsettings.loadModule(subCategoryView.model.index(index, 0));
}
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment