Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Global Search: Clicking an entry in the search window should lead to a selection in the corresponding library #9362

Closed
3 of 5 tasks
koppor opened this issue Nov 7, 2022 · 16 comments · Fixed by #9718, #9722 or #11010
Closed
3 of 5 tasks
Assignees
Labels
good first issue An issue intended for project-newcomers. Varies in difficulty. search ui

Comments

@koppor
Copy link
Member

koppor commented Nov 7, 2022

Issues left:

  • Preview in dark theme when no search results is still "white"
    image
  • Location of the pin to top button
    image
  • 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)
  • Search dialog has to retain size (-> store size in preferences)
  • Debatable: If the columns such as status or relevant should NOT be shown, because one could theoretically change the value of them in the search window...

Originally posted by @Siedlerchr in #8054 (comment)

@koppor koppor added the ui label Nov 7, 2022
@Siedlerchr Siedlerchr moved this to Free to take in Good First Issues Nov 7, 2022
@Siedlerchr Siedlerchr added the good first issue An issue intended for project-newcomers. Varies in difficulty. label Nov 9, 2022
@DanielS01ss
Copy link

Hi
I would love to work on this task!

@ThiloteE ThiloteE moved this from Free to take to Reserved in Good First Issues Feb 3, 2023
@ThiloteE ThiloteE added the FirstTimeCodeContribution Triggers GitHub Greeter Workflow label Feb 3, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2023

As a general advice for newcomers: check out Contributing for a start. Also, guidelines for setting up a local workspace is worth having a look at.

Feel free to ask here at GitHub, if you have any issue related questions. If you have questions about how to setup your workspace use JabRef's Gitter chat. Try to open a (draft) pull-request early on, so that people can see you are working on the issue and so that they can see the direction the pull request is heading towards. This way, you will likely receive valuable feedback.

@ThiloteE ThiloteE removed the FirstTimeCodeContribution Triggers GitHub Greeter Workflow label Feb 3, 2023
@DanielS01ss DanielS01ss removed their assignment Feb 23, 2023
@KunalBainsla
Copy link

Hi there.
I would like to take this task.

@JawadTUE
Copy link
Contributor

Hi, can I be assigned to this issue? I would like to work on it for a university project.

@ThiloteE
Copy link
Member

How to reproduce:
image

@ThiloteE
Copy link
Member

@KunalBainsla are you still on it?
@JawadTUE, I would say if KunalBainsla will not respond within a day, go for it. Thank you :-)

@JawadTUE
Copy link
Contributor

We started working on it and managed to fix 2 of the sub-issues. We will open a (draft) pull request soon.

@ThiloteE ThiloteE assigned JawadTUE and unassigned KunalBainsla Mar 30, 2023
@koppor
Copy link
Member Author

koppor commented Mar 31, 2023

Since some issues are independent, you can create branch per issue and submit a PR for each of that branch. This enables faster reviewing and getting the fixes into JabRef's main code.

@koppor
Copy link
Member Author

koppor commented Apr 2, 2023

Not all PRs are mreged yet, thus I keep it open.

@koppor koppor reopened this Apr 2, 2023
@github-project-automation github-project-automation bot moved this from Done to In Progress in Good First Issues Apr 2, 2023
@calixtus
Copy link
Member

calixtus commented Apr 3, 2023

Oops, was marked as "fixes" in the other pr. Sry.

@github-project-automation github-project-automation bot moved this from In Progress to Done in Good First Issues Apr 5, 2023
@koppor koppor reopened this Apr 6, 2023
@github-project-automation github-project-automation bot moved this from Done to In Progress in Good First Issues Apr 6, 2023
@koppor
Copy link
Member Author

koppor commented Apr 6, 2023

Item 2 and 3 not solved yet. Therefore, reopened.

@koppor koppor removed the good first issue An issue intended for project-newcomers. Varies in difficulty. label Apr 6, 2023
@Siedlerchr
Copy link
Member

Part 2 is fixed. Part 3 not yet

@ThiloteE ThiloteE moved this from In Progress to Free to take in Good First Issues Apr 10, 2023
@koppor koppor changed the title Global Search UI Issues (follow-up) Global Search: Clicking an entry in the search window should lead to a selection in the corresponding library Apr 12, 2023
@ThiloteE ThiloteE added the good first issue An issue intended for project-newcomers. Varies in difficulty. label May 22, 2023
@derronli
Copy link

I'd like to give this a try.

I began looking at the issue, and was thinking the solution lies in GlobalSearchResultDialogue.java (Below)
image

Was wondering if anyone could point me in the right direction.

@ThiloteE ThiloteE moved this from Free to take to Reserved in Good First Issues Sep 13, 2023
@ThiloteE ThiloteE assigned derronli and unassigned JawadTUE Sep 13, 2023
@calixtus
Copy link
Member

I believe you need to inject the frame as a librarytabcontainer the search dialog and use some sort of a select entry method in the librarytab class.

@Siedlerchr
Copy link
Member

Another idea would to be to set the selectedEntries in the statemanager as we only have the database contextes in the SearchResultsTableDataModel

@derronli
Copy link

Sorry to bother, I've been working on this for the past week. I've read the documentation on Javafx MVVM pattern and I think I have a high level understanding of the UI. I'm struggling to understand using the stateManager and the structure of selections. In GlobalSearchResultDialog.java I've tried using the setSelectedEntries method of the stateManager in a few different ways with no success.
Could I ask for further explanation on how to continue or an existing example on selection handling? Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment