Close the search on execution only as appropriate

When pressing ENTER, in some cases, there is nothing to do.
In those cases, let's not close the search pane prematurely.
Furthermore, some actions do not want the search pane to close
in response to being executed. Let's respect that, too.

Thanks to Jan Eglinger for noticing.

Author: ctrueden

src/main/java/org/scijava/ui/swing/search/SwingSearchBar.java

@@ -307,8 +307,8 @@ private void search() {
 	/** Called when the user hits ENTER. */
 	private void run() {
 		assertDispatchThread();
-		searchPanel.execute();
-		reset();
+		final boolean closeSearch = searchPanel.execute();
+		if (closeSearch) reset();
 	}
 
 	private void reset() {
@@ -610,7 +610,7 @@ private Font smaller(final Font font, final int decrement) {
 		}
 
 		/** Executes the default search action. */
-		private void execute() {
+		private boolean execute() {
 			assertDispatchThread();
 
 			// Figure out which result to execute.
@@ -619,14 +619,16 @@ private void execute() {
 			if (selectedResult == null) {
 				// Nothing is selected; use the first result on the list.
 				final int firstResultIndex = firstResultIndex();
-				if (firstResultIndex < 0) return; // no results available
+				if (firstResultIndex < 0) return false; // no results available
 				result = result(firstResultIndex);
 			}
 			else result = selectedResult;
 
 			final List<SearchAction> actions = searchService.actions(result);
-			if (actions.isEmpty()) return;
-			threadService.run(() -> actions.get(0).run());
+			if (actions.isEmpty()) return false;
+			final SearchAction action = actions.get(0);
+			threadService.run(() -> action.run());
+			return action.closesSearch();
 		}
 
 		private void rebuild() {