Fix deleting bookmarks

This regressed in the toplevel view model change. I checked the command against the commanditem. derp.
2025-12-31 09:27:03 +01:00 · 2025-03-21 21:22:08 -05:00
1 changed files with 4 additions and 9 deletions
--- a/src/modules/cmdpal/Microsoft.CmdPal.UI.ViewModels/TopLevelCommandManager.cs
+++ b/src/modules/cmdpal/Microsoft.CmdPal.UI.ViewModels/TopLevelCommandManager.cs
@@ -133,16 +133,11 @@ public partial class TopLevelCommandManager : ObservableObject,
            var wrapper = clone[i];
            try
            {
-                // TODO! this can be safer, we're not directly exposing ICommandItem's out of CPW anymore
-                var thisCommand = wrapper.ItemViewModel.Model.Unsafe;
-                if (thisCommand != null)
+                var isTheSame = wrapper == firstCommand;
+                if (isTheSame)
                {
-                    var isTheSame = thisCommand == firstCommand;
-                    if (isTheSame)
-                    {
-                        startIndex = i;
-                        break;
-                    }
+                    startIndex = i;
+                    break;
                }
            }
            catch