From c827d38f39e38fd0cf4f4ddfd2edf15ed0ebee36 Mon Sep 17 00:00:00 2001 From: zadam Date: Mon, 17 Jun 2019 22:25:22 +0200 Subject: [PATCH] when single note is selected and it's the same that was right clicked for context menu, then we treat it as no-selection action --- src/public/javascripts/services/tree_context_menu.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/public/javascripts/services/tree_context_menu.js b/src/public/javascripts/services/tree_context_menu.js index e96795cee..e02bd162a 100644 --- a/src/public/javascripts/services/tree_context_menu.js +++ b/src/public/javascripts/services/tree_context_menu.js @@ -33,7 +33,13 @@ class TreeContextMenu { const parentNote = await treeCache.getNote(branch.parentNoteId); const isNotRoot = note.noteId !== 'root'; const isHoisted = note.noteId === await hoistedNoteService.getHoistedNoteId(); - const noSelectedNotes = treeService.getSelectedNodes().length === 0; + + // some actions don't support multi-note so they are disabled when notes are selected + // the only exception is when the only selected note is the one that was right-clicked, then + // it's clear what the user meant to do. + const selNodes = treeService.getSelectedNodes(); + const noSelectedNotes = selNodes.length === 0 + || (selNodes.length === 1 && selNodes[0] === this.node); const insertNoteAfterEnabled = isNotRoot && !isHoisted && parentNote.type !== 'search'; const insertChildNoteEnabled = note.type !== 'search';