Distinguish saved group spacing from semantic reorder in UI guidance

This commit is contained in:
2026-09-24 16:05:45 -07:00
parent 4a27c8ef0b
commit 8d6fac4701
4 changed files with 7 additions and 4 deletions
+1 -1
View File
@@ -1377,7 +1377,7 @@ impl App {
Message::Command(Command::MoveStep(..)) | Message::Delete Message::Command(Command::MoveStep(..)) | Message::Delete
) )
{ {
self.status = "Group movement and removal need a whole-selection operation. Select one occurrence to edit it individually.".into(); self.status = "Group reordering and removal need a whole-selection operation. Select one occurrence to edit it individually.".into();
return Task::none(); return Task::none();
} }
let preview_read_only = self.process_preview_primary().is_some(); let preview_read_only = self.process_preview_primary().is_some();
+1 -1
View File
@@ -2387,7 +2387,7 @@ impl canvas::Program<Message> for Scene {
} }
let intent = if press.moved { let intent = if press.moved {
Intent::Refuse( Intent::Refuse(
"Group movement is being implemented; select one occurrence to edit it" "This selection cannot be moved here; select one occurrence for its individual actions"
.into(), .into(),
) )
} else { } else {
@@ -278,7 +278,10 @@ pub fn view<'a>(app: &'a App, owner: Id, out: Column<'a, Message>) -> Element<'a
let entries = [ let entries = [
( (
"S", "S",
if selected_count > 1 { if selected_count > 1 && matches!(context.representation, Representation::Saved { .. })
{
"Drag messages vertically or lifelines horizontally; Shift-click to change selection"
} else if selected_count > 1 {
"Shift-click to change selection; click a member to select only it" "Shift-click to change selection; click a member to select only it"
} else if matches!(context.representation, Representation::Saved { .. }) { } else if matches!(context.representation, Representation::Saved { .. }) {
"Select / move messages and lifelines" "Select / move messages and lifelines"
+1 -1
View File
@@ -367,7 +367,7 @@ pub fn popup(app: &App) -> Option<Menu<Message>> {
snapshot.targets().len() snapshot.targets().len()
))); )));
entries.push(Entry::disabled( entries.push(Entry::disabled(
"Move selected occurrences (not yet available)", "Reorder selected occurrences (not yet available)",
)); ));
entries.push(Entry::disabled( entries.push(Entry::disabled(
"Remove selected occurrences (not yet available)", "Remove selected occurrences (not yet available)",