notedeck

commit 9aa306bb3b1977bef1181258ef7830f72bd9612d
parent 15b8656d1f1d66d23fc7e5f4603d097d7ff98ad6
Author: William Casarin <jb55@jb55.com>
Date:   Thu, 19 Feb 2026 09:39:54 -0800

dave: restore shift highlight on permission message link

Show "+ msg [⇧]" instead of "+ msg" and highlight in warn color
when Shift is held, restoring the visual feedback for desktop users
that was lost during the tentative send refactor.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Diffstat:
M
crates/notedeck_dave/src/ui/dave.rs
 | 
20
+++++++++++---------

1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/crates/notedeck_dave/src/ui/dave.rs b/crates/notedeck_dave/src/ui/dave.rs
@@ -691,7 +691,7 @@ impl<'a> DaveUi<'a> {
                     }
                 }
 
-                add_msg_link(ui, action);
+                add_msg_link(ui, shift_held, action);
             }
         });
     }
@@ -800,7 +800,7 @@ impl<'a> DaveUi<'a> {
                                 }
                             }
 
-                            add_msg_link(ui, &mut action);
+                            add_msg_link(ui, shift_held, &mut action);
                         }
                     });
                 });
@@ -1204,14 +1204,16 @@ fn tentative_send_ui(
     }
 }
 
-/// Clickable "+ msg" link that enters tentative accept mode.
-fn add_msg_link(ui: &mut egui::Ui, action: &mut Option<DaveAction>) {
+/// Clickable "+ msg [⇧]" link that enters tentative accept mode.
+/// Highlights in warn color when Shift is held on desktop.
+fn add_msg_link(ui: &mut egui::Ui, shift_held: bool, action: &mut Option<DaveAction>) {
+    let color = if shift_held {
+        ui.visuals().warn_fg_color
+    } else {
+        ui.visuals().weak_text_color()
+    };
     if ui
-        .link(
-            egui::RichText::new("+ msg")
-                .color(ui.visuals().weak_text_color())
-                .small(),
-        )
+        .link(egui::RichText::new("+ msg [⇧]").color(color).small())
         .clicked()
     {
         *action = Some(DaveAction::TentativeAccept);