From 6b468f9c4ab87c956d04d3ecd23a2f816f459bd5 Mon Sep 17 00:00:00 2001 From: Chris Bobbe Date: Wed, 4 Sep 2024 00:11:05 -0700 Subject: [PATCH] msglist: Fix mark-as-read button icon color in dark mode After sweeping through the message-list screen to implement dark theme (#843), in late July, the Flutter framework merged flutter/flutter#143501, in early August. This made it so our `foregroundColor` no longer controlled the icon color, as promised in the dartdocs of `foregroundColor` and `iconColor`. I commented about this on the PR: https://github.com/flutter/flutter/pull/143501#issuecomment-2328083560 Fixes: #926 --- lib/widgets/message_list.dart | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/widgets/message_list.dart b/lib/widgets/message_list.dart index f5088b1fc7..507e5eda73 100644 --- a/lib/widgets/message_list.dart +++ b/lib/widgets/message_list.dart @@ -758,6 +758,7 @@ class _MarkAsReadWidgetState extends State { // state is disabled, pressed, etc. We handle those states // separately, via MarkAsReadAnimation. foregroundColor: const WidgetStatePropertyAll(Colors.white), + iconColor: const WidgetStatePropertyAll(Colors.white), backgroundColor: WidgetStatePropertyAll(messageListTheme.unreadMarker), ), onPressed: _loading ? null : () => _handlePress(context),