Merge #4495 Remove dependencies from changeset when skipping a change

Author: HebaruSan

CHANGELOG.md

@@ -13,6 +13,7 @@ All notable changes to this project will be documented in this file.
 - [Core] Ensure encoding is UTF-8 when saving JSON (#4475, #4478 by: HebaruSan)
 - [GUI] Display release date in local time zone (#4488 by: HebaruSan)
 - [GUI] Fix freezing after startup with empty mod list (#4491, #4492 by: HebaruSan)
+- [GUI] Remove dependencies from changeset when skipping a change (#4495 by: HebaruSan)
 
 ### Internal
 

GUI/Controls/Changeset.cs

@@ -81,8 +81,8 @@ private void ChangesGrid_CellClick(object sender, DataGridViewCellEventArgs e)
                 && row.ConfirmUncheck())
             {
                 (ChangesGrid.DataSource as BindingList<ChangesetRow>)?.Remove(row);
-                changeset?.Remove(row.Change);
                 OnRemoveItem?.Invoke(row.Change);
+                changeset?.Remove(row.Change);
             }
         }
 

GUI/Controls/ManageMods.cs

@@ -1121,6 +1121,18 @@ public void RemoveChangesetItem(ModChange change)
                     guiMod.SelectedMod = guiMod.InstalledMod?.Module;
                     switch (change.ChangeType)
                     {
+                        case GUIModChangeType.Remove:
+                            if (row.Cells[Installed.Index] is DataGridViewCheckBoxCell instCell)
+                            {
+                                instCell.Value = true;
+                            }
+                            break;
+                        case GUIModChangeType.Install:
+                            if (row.Cells[Installed.Index] is DataGridViewCheckBoxCell rmCell)
+                            {
+                                rmCell.Value = false;
+                            }
+                            break;
                         case GUIModChangeType.Replace:
                             if (row.Cells[ReplaceCol.Index] is DataGridViewCheckBoxCell checkCell)
                             {