feat(*): use error color on delete action in notice card menu

realth000 · Dec 28, 2024 · 020ff29 · 020ff29
1 parent 1c658ef
commit 020ff29
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 4 deletions.
diff --git a/lib/utils/show_dialog.dart b/lib/utils/show_dialog.dart
@@ -42,6 +42,7 @@ Future<bool?> showQuestionDialog({
   required String title,
   String? message,
   TextSpan? richMessage,
+  bool dangerous = false,
 }) async {
   assert(
     message != null || richMessage != null,
@@ -64,9 +65,17 @@ Future<bool?> showQuestionDialog({
             },
           ),
           TextButton(
-            child: Text(context.t.general.ok),
+            child: Text(
+              context.t.general.ok,
+              style: dangerous
+                  ? TextStyle(color: Theme.of(context).colorScheme.error)
+                  : null,
+            ),
             onPressed: () {
-              Navigator.pop(context, true);
+              Navigator.pop(
+                context,
+                true,
+              );
             },
           ),
         ],

diff --git a/lib/widgets/card/notice_card_v2.dart b/lib/widgets/card/notice_card_v2.dart
@@ -128,9 +128,17 @@ class _NoticeCardV2State extends State<NoticeCardV2> {
                   value: _Actions.deleteItem,
                   child: Row(
                     children: [
-                      const Icon(Icons.delete_forever_outlined),
+                      Icon(
+                        Icons.delete_forever_outlined,
+                        color: Theme.of(context).colorScheme.error,
+                      ),
                       sizedBoxPopupMenuItemIconSpacing,
-                      Text(tr.delete.title),
+                      Text(
+                        tr.delete.title,
+                        style: TextStyle(
+                          color: Theme.of(context).colorScheme.error,
+                        ),
+                      ),
                     ],
                   ),
                 ),
@@ -147,6 +155,7 @@ class _NoticeCardV2State extends State<NoticeCardV2> {
                       context: context,
                       title: tr.title,
                       message: tr.detail,
+                      dangerous: true,
                     );
                     if (!context.mounted || result == null || !result) {
                       return;