Skip to content

Commit 50892f7

Browse files
committed
ux: re-order context menu items for selected multiple branches
1 parent fec13cd commit 50892f7

File tree

1 file changed

+11
-9
lines changed

1 file changed

+11
-9
lines changed

src/Views/BranchTree.axaml.cs

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -399,15 +399,6 @@ private void OnTreeContextRequested(object _1, ContextRequestedEventArgs _2)
399399
else if (branches.Find(x => x.IsCurrent) == null)
400400
{
401401
var menu = new ContextMenu();
402-
var deleteMulti = new MenuItem();
403-
deleteMulti.Header = App.Text("BranchCM.DeleteMultiBranches", branches.Count);
404-
deleteMulti.Icon = App.CreateMenuIcon("Icons.Clear");
405-
deleteMulti.Click += (_, ev) =>
406-
{
407-
repo.DeleteMultipleBranches(branches, branches[0].IsLocal);
408-
ev.Handled = true;
409-
};
410-
menu.Items.Add(deleteMulti);
411402

412403
var mergeMulti = new MenuItem();
413404
mergeMulti.Header = App.Text("BranchCM.MergeMultiBranches", branches.Count);
@@ -418,6 +409,17 @@ private void OnTreeContextRequested(object _1, ContextRequestedEventArgs _2)
418409
ev.Handled = true;
419410
};
420411
menu.Items.Add(mergeMulti);
412+
menu.Items.Add(new MenuItem() { Header = "-" });
413+
414+
var deleteMulti = new MenuItem();
415+
deleteMulti.Header = App.Text("BranchCM.DeleteMultiBranches", branches.Count);
416+
deleteMulti.Icon = App.CreateMenuIcon("Icons.Clear");
417+
deleteMulti.Click += (_, ev) =>
418+
{
419+
repo.DeleteMultipleBranches(branches, branches[0].IsLocal);
420+
ev.Handled = true;
421+
};
422+
menu.Items.Add(deleteMulti);
421423

422424
menu?.Open(this);
423425
}

0 commit comments

Comments
 (0)