From d2892f90765a44395a8c507b8add39d53c6f6d73 Mon Sep 17 00:00:00 2001
From: Michael Bucari-Tovo <mbucari1@gmail.com>
Date: Thu, 2 Sep 2021 11:11:40 -0600
Subject: [PATCH 1/2] Fix crash when checkbox checked via spacebar.

---
 LibationWinForms/Dialogs/RemoveBooksDialog.cs | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/LibationWinForms/Dialogs/RemoveBooksDialog.cs b/LibationWinForms/Dialogs/RemoveBooksDialog.cs
index d7fb7c09..37e7bbb7 100644
--- a/LibationWinForms/Dialogs/RemoveBooksDialog.cs
+++ b/LibationWinForms/Dialogs/RemoveBooksDialog.cs
@@ -7,6 +7,7 @@ using System.Linq;
 using System.Windows.Forms;
 using ApplicationServices;
 using DataLayer;
+using Dinah.Core.Threading;
 using Dinah.Core.DataBinding;
 using InternalUtilities;
 using LibationWinForms.Login;
@@ -32,9 +33,8 @@ namespace LibationWinForms.Dialogs
 			InitializeComponent();
 			_labelFormat = label1.Text;
 
-			_dataGridView.CellContentClick += (s, e) => _dataGridView.CommitEdit(DataGridViewDataErrorContexts.Commit);
-			_dataGridView.CellValueChanged += DataGridView1_CellValueChanged;
-			_dataGridView.BindingContextChanged += (s, e) => UpdateSelection();
+			_dataGridView.CellContentClick +=(s,e) => _dataGridView.CommitEdit(DataGridViewDataErrorContexts.Commit);
+			_dataGridView.BindingContextChanged += _dataGridView_BindingContextChanged;
 
 			var orderedGridEntries = _libraryBooks
 				.Select(lb => new RemovableGridEntry(lb))
@@ -47,10 +47,10 @@ namespace LibationWinForms.Dialogs
 			_dataGridView.Enabled = false;
 		}
 
-		private void DataGridView1_CellValueChanged(object sender, DataGridViewCellEventArgs e)
+		private void _dataGridView_BindingContextChanged(object sender, EventArgs e)
 		{
-			if (e.ColumnIndex == 0)
-				UpdateSelection();
+			_dataGridView.Sort(_dataGridView.Columns[0], ListSortDirection.Descending);
+			UpdateSelection();
 		}
 
 		private async void RemoveBooksDialog_Shown(object sender, EventArgs e)
@@ -124,7 +124,6 @@ namespace LibationWinForms.Dialogs
 		}
 		private void UpdateSelection()
 		{
-			_dataGridView.Sort(_dataGridView.Columns[0], ListSortDirection.Descending);
 			var selectedCount = SelectedCount;
 			label1.Text = string.Format(_labelFormat, selectedCount, selectedCount != 1 ? "s" : string.Empty);
 			btnRemoveBooks.Enabled = selectedCount > 0;

From 6d2dff1a9859173fa4dd219f71b6264ffe3d8ea8 Mon Sep 17 00:00:00 2001
From: Michael Bucari-Tovo <mbucari1@gmail.com>
Date: Thu, 2 Sep 2021 11:21:20 -0600
Subject: [PATCH 2/2] Code Cleanup

---
 LibationWinForms/Dialogs/RemoveBooksDialog.cs | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/LibationWinForms/Dialogs/RemoveBooksDialog.cs b/LibationWinForms/Dialogs/RemoveBooksDialog.cs
index 37e7bbb7..90c3283c 100644
--- a/LibationWinForms/Dialogs/RemoveBooksDialog.cs
+++ b/LibationWinForms/Dialogs/RemoveBooksDialog.cs
@@ -1,16 +1,15 @@
-using System;
+using ApplicationServices;
+using DataLayer;
+using Dinah.Core.DataBinding;
+using InternalUtilities;
+using LibationWinForms.Login;
+using System;
 using System.Collections;
 using System.Collections.Generic;
 using System.ComponentModel;
 using System.Data;
 using System.Linq;
 using System.Windows.Forms;
-using ApplicationServices;
-using DataLayer;
-using Dinah.Core.Threading;
-using Dinah.Core.DataBinding;
-using InternalUtilities;
-using LibationWinForms.Login;
 
 namespace LibationWinForms.Dialogs
 {
@@ -33,7 +32,7 @@ namespace LibationWinForms.Dialogs
 			InitializeComponent();
 			_labelFormat = label1.Text;
 
-			_dataGridView.CellContentClick +=(s,e) => _dataGridView.CommitEdit(DataGridViewDataErrorContexts.Commit);
+			_dataGridView.CellContentClick += (s, e) => _dataGridView.CommitEdit(DataGridViewDataErrorContexts.Commit);
 			_dataGridView.BindingContextChanged += _dataGridView_BindingContextChanged;
 
 			var orderedGridEntries = _libraryBooks