releases/ClaudeDo
2
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases 3 Wiki Activity

refactor(ui): harden double-click edit handlers

Browse Source 
Remove redundant SelectedList/SelectedTask assignments (single-tap already
updated selection via binding) and CanExecute guards (always true for
unconditional [RelayCommand]). Set e.Handled = true to stop DoubleTapped
from bubbling to the ListBox.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Mika Kuns
2026-04-13 15:04:19 +02:00
parent db5a447b12
commit 6727cc4e9d
2 changed files with 6 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/ClaudeDo.Ui/Views/MainWindow.axaml.cs
View File

@@ -20,12 +20,11 @@ public partial class MainWindow : Window
private void OnListItemDoubleTapped(object? sender, TappedEventArgs e)
{
if (sender is Control c && c.DataContext is ListItemViewModel item
if (sender is Control c && c.DataContext is ListItemViewModel
&& DataContext is MainWindowViewModel vm)
{
vm.SelectedList = item;
if (vm.EditListCommand.CanExecute(null))
vm.EditListCommand.Execute(null);
e.Handled = true;
}
}
}

5
src/ClaudeDo.Ui/Views/TaskListView.axaml.cs
View File

@@ -13,12 +13,11 @@ public partial class TaskListView : UserControl
private void OnTaskItemDoubleTapped(object? sender, TappedEventArgs e)
{
if (sender is Control c && c.DataContext is TaskItemViewModel item
if (sender is Control c && c.DataContext is TaskItemViewModel
&& DataContext is TaskListViewModel vm)
{
vm.SelectedTask = item;
if (vm.EditTaskCommand.CanExecute(null))
vm.EditTaskCommand.Execute(null);
e.Handled = true;
}
}
}