From 0bcb8fdecfa66f255503ec05eff865ea70960071 Mon Sep 17 00:00:00 2001 From: Penlane Date: Sat, 25 Jul 2026 17:46:58 +0200 Subject: [PATCH] feat(mobile): clean up timeline filter dialog (2 icons instead of checkmark) --- mobile/lib/presentation/pages/dev/main_timeline.page.dart | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/mobile/lib/presentation/pages/dev/main_timeline.page.dart b/mobile/lib/presentation/pages/dev/main_timeline.page.dart index 4f2dccef69..47cc7c5583 100644 --- a/mobile/lib/presentation/pages/dev/main_timeline.page.dart +++ b/mobile/lib/presentation/pages/dev/main_timeline.page.dart @@ -88,6 +88,7 @@ class _AssetOriginFilterButton extends ConsumerWidget { AssetOriginFilter.all, 'timeline_filter_all'.tr(), Icons.photo_library_outlined, + Icons.photo_library, currentSelection, ), _buildItem( @@ -96,6 +97,7 @@ class _AssetOriginFilterButton extends ConsumerWidget { AssetOriginFilter.remote, 'timeline_filter_remote_only'.tr(), Icons.cloud_outlined, + Icons.cloud, currentSelection, ), _buildItem( @@ -104,6 +106,7 @@ class _AssetOriginFilterButton extends ConsumerWidget { AssetOriginFilter.localOnly, 'timeline_filter_local_only'.tr(), Icons.cloud_off_outlined, + Icons.cloud_off, currentSelection, ), ], @@ -116,6 +119,7 @@ class _AssetOriginFilterButton extends ConsumerWidget { AssetOriginFilter staticFilter, String label, IconData icon, + IconData iconActive, AssetOriginFilter currentFilter, ) { final isSelected = staticFilter == currentFilter; @@ -124,7 +128,7 @@ class _AssetOriginFilterButton extends ConsumerWidget { return MenuItemButton( onPressed: () => ref.read(assetOriginFilterProvider.notifier).state = staticFilter, child: ListTile( - leading: Icon(icon, color: isSelected ? primaryColor : null), + leading: Icon(isSelected ? iconActive : icon, color: isSelected ? primaryColor : null), title: Text( label, style: context.textTheme.bodyLarge?.copyWith( @@ -132,7 +136,6 @@ class _AssetOriginFilterButton extends ConsumerWidget { color: isSelected ? primaryColor : null, ), ), - trailing: isSelected ? Icon(Icons.check, color: primaryColor) : null, selected: isSelected, selectedColor: primaryColor, ),