From 89e8c5d9f743ba6b80ccd7d7d9789d1d1d071c73 Mon Sep 17 00:00:00 2001 From: Damien De Paoli Date: Tue, 28 Oct 2025 22:31:15 +1100 Subject: [PATCH] fix BUG-125: right-click changes highlight if its not in the highlight set --- BUGs | 4 ---- internal/js/files_support.js | 6 ++++++ 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/BUGs b/BUGs index 7f8209e..313fd5d 100644 --- a/BUGs +++ b/BUGs @@ -1,10 +1,6 @@ ### Next: 146 BUG-143: if I skip next fast enough in files_ip, it can get the warning about changes to entry list... (would have though I disabled next page until loaded) BUG-140: When db is restarted underneath PA, it crashes job mgr... It should just accept timeouts, and keep trying to reconnect every 2? mins -BUG-125: when an image is highlighted, then post the contextmenu on a different image - the highlight does not move to the new image - and the selected menu function processes the original or the new depending on the way the code works. - There is a chance we need to change the document on click to a mouse down (or whatever the context menu - uses for default), rather than just fix the highlight BUG-118: can move files from Bin path, but it leaves the del_file entry for it - need to remove it BUG-117: when search returns files that can be deleted and/or restored, the icon stays as delete and tries to delete! BUG-106: cant add trudy /pat? as refimgs via FaceDBox diff --git a/internal/js/files_support.js b/internal/js/files_support.js index 4fa6439..d49d41f 100644 --- a/internal/js/files_support.js +++ b/internal/js/files_support.js @@ -916,6 +916,12 @@ $.contextMenu({ selector: '.entry', itemClickEvent: "click", build: function($triggerElement, e) { + // if we are not in the highlight set, then move the highlight to this element + if( ! $(e.currentTarget).is('.highlight') ) + { + $('.highlight').removeClass('highlight'); + $(e.currentTarget).addClass('highlight') + } // when right-clicking & no selection add one OR deal with ctrl/shift right-lick as it always changes seln if( NoSel() || e.ctrlKey || e.shiftKey ) {