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 UI issues #8054

Closed
3 of 4 tasks
koppor opened this issue Sep 2, 2021 · 8 comments
Closed
3 of 4 tasks

Global Search UI issues #8054

koppor opened this issue Sep 2, 2021 · 8 comments

Comments

@koppor
Copy link
Member

koppor commented Sep 2, 2021

#8045 introduced global search. There are some open points:

@koppor koppor added this to the v5.4 milestone Sep 2, 2021
@mlep
Copy link
Contributor

mlep commented Sep 3, 2021

I am happy to see that the global search is back! Very useful feature!

A few glitches (some grain to grind 😉 ):

UI

  • If global search is NOT activated, the button Search across libraries window still works for the current library (which is great!). So, I guess it would be better described as Display results in an external window (or something like this)
  • Using dark theme: in the external window, the preview area is white if no entry is selected (then it becomes dark grey once an entry is selected). The UI would be more consistent if the preview area was dark grey from the beginning.
  • In the external window, the button Keep dialog always on top is located in a weird place. Maybe it would be better if is was moved lower, on the same line as the button OK
  • The UI does not retain the size of the Search across libraries window. Open it, changes its size, close it, open it again : it has its original size.
  • The Search across libraries window does not have a title.
  • The columns shown in the external window should be the same as in the entry tables of the files. Currently, columns such as Read status and Relevance are not displayed in the external window

Functioning

  • Opening another file should not reset a global search. Open a file, activate global search and enter a string to search, open a second file: the global search is deactivated and the search string is missing in the first file. Note: the global search is not reset when closing a file (which is fine!)
  • When deactivating Global search, the external window continue to display the results of the global search. Open 2 files, enter a search string (that gives entries for each file --- let's say 10 + 5), click on global search (at this point the external window display the 15 entries, which is OK), close the external window, unselect the global search, open the external window: 15 entries are still shown (while it should be 5 or 10, depending on the selected file).
  • I also came across a raised exception (related to the marked entries), but cannot reproduce it (yet)...

@Siedlerchr Siedlerchr added type: enhancement bug Confirmed bugs or reports that are very likely to be bugs labels Sep 3, 2021
@Siedlerchr
Copy link
Member

Thanks for the detailed feedback @mlep Regarding the special fields to display @calixtus and I decided not to show them, because it does make sense that you can modify them. Could also produce some side effects...

@Siedlerchr
Copy link
Member

I found the issue with the library field. Totally stupid. As I previously tested the ugly hack, it modified the entries and added the library field. Normally the library field is a biblatex field for a different purpose.

I will look into a different way to get the library information

@Siedlerchr
Copy link
Member

Siedlerchr commented Sep 9, 2021

@koppor

(Optional) the places where the search hit should be highlighted
(we had this functionality in JabRef 2.9 beta -

The functionality is also implemented in the current versions since a long time in the Preview window

@koppor
Copy link
Member Author

koppor commented Sep 27, 2021

@koppor
Copy link
Member Author

koppor commented Oct 25, 2021

* If global search is NOT activated, the button `Search across libraries window` still works for the current library (which is great!). So, I guess it would be better described as `Display results in an external window` (or something like this)

The world icon denotes something different than the global search.

The first one "just" keeps the search string when switching tabs, the second one searches globally

grafik

Idea: change to an ice berg / mountain.

* Using dark theme: in the external window, [...]

This is when searching for something without results

grafik

* In the external window, the button `Keep dialog always on top` is located in a weird place. Maybe it would be better if is was moved lower, on the same line as the button `OK`

OK :)

grafik

Siedlerchr added a commit that referenced this issue Nov 15, 2021
@koppor koppor added ui and removed bug Confirmed bugs or reports that are very likely to be bugs labels Nov 22, 2021
@koppor koppor removed this from the v5.4 milestone Nov 22, 2021
Siedlerchr added a commit that referenced this issue Nov 28, 2021
* Do not resize main table columns in search dialog window

Follow up from #8134
Fixes part of #8054

* checkstyle

* Refactor and store Search Dialog Column changes

* checkstyle

* refactor, get rid of duplicate column preferences

* fix checkstlye again

* Revert reformatting hazard

* fix checkstyle

* Add a new column type library and check if it exists before adding to search reults

* Remove library col from main table...

* fix checkstyle

* Reverted unnecessary reformatting

* fix checkstyle

Co-authored-by: Carl Christian Snethlage <cc.snethlage@gmail.com>
Siedlerchr added a commit that referenced this issue Dec 3, 2021
@Siedlerchr
Copy link
Member

Siedlerchr commented Dec 3, 2021

Current status after #8294 is merged

  • Window has a title
  • Search string is applied when opening a new library and the option "keep search string across libraries" is selected

Only open issues left:

  • Preview in dark theme when no search results is still "white"
  • Location of the pin to top button
  • Debatable: If the columns like status or relevant should be shown, because you could theoretically change the value of them in the search window...
  • When clicking an entry in the external window, it should be marked in the corresponding library (e.g. select the library, select the item in the main table)
  • Search dialog does not retain size

Siedlerchr added a commit that referenced this issue Dec 5, 2021
@Siedlerchr Siedlerchr changed the title Global Search issues Global Search UI issues Dec 22, 2021
@koppor
Copy link
Member Author

koppor commented Nov 7, 2022

We opened a follow-up issue to be able to focus on the open points.

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

No branches or pull requests

3 participants