Selected entry in search window selects entry in the main table

[LoayGhreeb]

Closes #9362

When clicking an entry in the search window, it should be marked in the corresponding library (JabRef should focus the library and then select the item in the main table)

preview.mp4

Mandatory checks

• Change in CHANGELOG.md described in a way that is understandable for the average user (if applicable)
• Tests created for changes (if applicable)
• Manually tested changed features in running JabRef (always required)
• Screenshots added in PR description (for UI changes)
• Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
• Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

[LoayGhreeb]

I figured out two issues with the current implementation:

• When using setCurrentTab method, it changes the activeDatabase. This change maybe trigger a listener associated with activeDatabase, leading to unexpected behavior in the resultsTable.

  issue.mp4

• After closing the search result window, the search query remains in the search bar. When trying to edit the query, the resultsTable selection item listener still works, indicating that the resultsTable is still in memory. (Before this PR, this behavior occurred but didn't affect anything because changes were only in the previewViewer).

  resultsTable.getSelectionModel().selectedItemProperty().addListener((obs, old, newValue) -> {
              if (newValue != null) {
                  previewViewer.setEntry(newValue.getEntry());
              } else {
                  previewViewer.setEntry(old.getEntry());
              }
          });

[koppor]

Small comment: In future, try to separate these small side changes to other pull requests. Reason: If a PR takes long to review, these kind of changes will be "outdated" (e.g., causing conflicts, because other PRs also touch these places).

On Linux (and Mac) you can try GitButler for "carving out" unrelated changes.

[LoayGhreeb]

On Linux (and Mac) you can try GitButler for "carving out" unrelated changes.

GitButler is now supported on Windows, but for some reason it didn't work with JabRef 😅

[Siedlerchr]

lgtm!