From daaffb6bd5b239045c622a4e5ee4d6a12e680ef1 Mon Sep 17 00:00:00 2001
From: Elias Bonnici <elias.bonnici@yubico.com>
Date: Fri, 22 Mar 2024 13:39:14 +0100
Subject: [PATCH] Change selected colors in `AppListItem`

---
 lib/app/views/app_list_item.dart | 5 +++++
 lib/piv/views/piv_screen.dart    | 5 ++++-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/lib/app/views/app_list_item.dart b/lib/app/views/app_list_item.dart
index cb288662..394aaa00 100644
--- a/lib/app/views/app_list_item.dart
+++ b/lib/app/views/app_list_item.dart
@@ -70,6 +70,7 @@ class _AppListItemState<T> extends ConsumerState<AppListItem> {
     final doubleTapIntent = widget.doubleTapIntent;
     final trailing = widget.trailing;
     final hasFeature = ref.watch(featureProvider);
+    final colorScheme = Theme.of(context).colorScheme;
 
     return Semantics(
       label: widget.semanticTitle ?? widget.title,
@@ -124,6 +125,10 @@ class _AppListItemState<T> extends ConsumerState<AppListItem> {
               ListTile(
                 mouseCursor:
                     widget.tapIntent != null ? SystemMouseCursors.click : null,
+                shape: RoundedRectangleBorder(
+                    borderRadius: BorderRadius.circular(48)),
+                selectedTileColor: colorScheme.secondaryContainer,
+                selectedColor: colorScheme.onSecondaryContainer,
                 selected: widget.selected,
                 leading: widget.leading,
                 title: Text(
diff --git a/lib/piv/views/piv_screen.dart b/lib/piv/views/piv_screen.dart
index 573b54f3..66e61e27 100644
--- a/lib/piv/views/piv_screen.dart
+++ b/lib/piv/views/piv_screen.dart
@@ -249,7 +249,10 @@ class _CertificateListItem extends ConsumerWidget {
       leading: CircleAvatar(
         foregroundColor: colorScheme.onSecondary,
         backgroundColor: colorScheme.secondary,
-        child: Icon(slot.isRetired ? Symbols.manage_history : Symbols.badge),
+        child: Icon(
+          slot.isRetired ? Symbols.manage_history : Symbols.badge,
+          fill: selected ? 1 : 0,
+        ),
       ),
       title: slot.getDisplayName(l10n),
       subtitle: certInfo != null