fix: selected node not cleared when a different node is selected

Author: ycnmhd

src/stores/view/subscriptions/actions/maybe-clear-selection.ts

@@ -4,7 +4,6 @@ import {
     getViewEventType,
     ViewEventType,
 } from 'src/stores/view/helpers/get-view-event-type';
-import { maybeClearSelection } from 'src/stores/view/subscriptions/actions/maybe-clear-selection';
 import { updateSearchResults } from 'src/stores/view/subscriptions/actions/update-search-results';
 import { focusContainer } from 'src/stores/view/subscriptions/effects/focus-container';
 import { persistActiveNodeInPluginSettings } from 'src/stores/view/subscriptions/actions/persist-active-node-in-plugin-settings';
@@ -49,15 +48,6 @@ export const onViewStateUpdate = (
         }
     }
 
-    if (
-        activeNodeChange &&
-        activeNodeHasChanged &&
-        type !== 'DOCUMENT/NAVIGATE_USING_KEYBOARD' &&
-        type !== 'DOCUMENT/JUMP_TO_NODE'
-    ) {
-        maybeClearSelection(view, action);
-    }
-
     if (action.type === 'SEARCH/SET_QUERY') {
         updateSearchResults(view);
     }

src/stores/view/subscriptions/on-view-state-update.ts

@@ -41,6 +41,8 @@ const updateDocumentState = (
         action.type === 'view/set-active-node/search'
     ) {
         updateActiveNode(state.document, action.payload.id, state);
+        if (!state.document.selectedNodes.has(state.document.activeNode))
+            resetSelectionState(state.document);
     } else if (action.type === 'DOCUMENT/NAVIGATE_USING_KEYBOARD') {
         navigateUsingKeyboard(state.document, state, action, context.columns);
     } else if (action.type === 'SEARCH/SET_QUERY') {

src/stores/view/view-reducer.ts

@@ -37,7 +37,7 @@ export const handleEscapeKey = (view: LineageView) => {
             type: 'SEARCH/TOGGLE_INPUT',
         });
         return true;
-    } else if (selection.size > 1) {
+    } else if (selection.size > 0) {
         viewStore.dispatch({
             type: 'DOCUMENT/CLEAR_SELECTION',
         });