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

Searching: extension to stop filtering #444

Closed
lecc0r opened this issue Dec 2, 2015 · 15 comments
Closed

Searching: extension to stop filtering #444

lecc0r opened this issue Dec 2, 2015 · 15 comments

Comments

@lecc0r
Copy link

lecc0r commented Dec 2, 2015

The newly implemented search function is great and helps a lot to search/filter my 3000 paper data base. However, after having done my search I want to see again all entries. Therefore, I have to stop the searching/filtering by either clicking into the search field or press Ctrl + F and delete the search term.
I would like to suggest a handy extension to stop the searching/filtering: either a stop button next to the search field or alternatively stop the searching/filtering by hitting the Escape button on the keyboard. Both would be preferable.

@simonharrer
Copy link
Contributor

Will implement this for the Escape button.

@simonharrer simonharrer self-assigned this Dec 2, 2015
@tobiasdiez
Copy link
Member

What about displaying a delete/clear icon in the search field, as for example here:
Image

@matthiasgeiger
Copy link
Member

@tobiasdiez Yes, that would be the perfect solution. Do you want to implement it?
Potential starting point might be this blogpost: https://tips4java.wordpress.com/2009/09/27/component-border/
A fitting icon would be https://materialdesignicons.com/icon/close-circle

@simonharrer
Copy link
Contributor

Good idea, but I do not want to have too much swing UI code before changing at some point in time to JavaFX. Hence, I aim for simple UI solutions. But as a user I would like to have such a feature.

@matthiasgeiger
Copy link
Member

The simpler, and not so fancy solution would be a simple button next to the search field ;-)

@stefan-kolb
Copy link
Member

ESC binding is still appropriate as additional option, like in Intellij.

@tobiasdiez
Copy link
Member

For the moment, I second the idea of a button next to the search field. I will gladly implement the inline icon after we switched to JavaFX.

@simonharrer
Copy link
Contributor

  • what should the button say: "Clear", "Reset", "Cancel", display an X?
  • where should the ESC behavior apply: only in the text field? when the focus is also in the main table?

@lecc0r
Copy link
Author

lecc0r commented Dec 3, 2015

  • I would go for "Clear"
  • Ideally it should apply to both. When searching for an entry I often get more than one result and I then have to click on the one I was searching for. Thus, the focus is often in the main table. But it should always be able to quickly clear the search field by hitting ESC when the focus is in the text field.

@simonharrer
Copy link
Contributor

At the moment, ESC is already bound on the main table. You can just enter letters and numbers when the main table is selected and it automatically should find the entry that starts with the entered letters and numbers. ESC ends this jumping around.

Hence, I would propose to only implement the button + ESC with focus in search field.

simonharrer added a commit that referenced this issue Dec 3, 2015
…utton or by pressing ESC with having focus in the search field.
@simonharrer
Copy link
Contributor

Please have a look at the current master build. It contains the implemented feature.

@lecc0r
Copy link
Author

lecc0r commented Dec 3, 2015

@simonharrer Yes, seems to work fine!

@koppor
Copy link
Member

koppor commented Dec 5, 2015

While working with current master, I found out that the current solution is disturbing for my workflow. I would have expected that the first "ESC" closes the autocomplete dropdown. The second "ESC" focues the main table and the third "ESC" clears the search.

I used to search for something (in floating mode), then press ESC and then use the cursor up- and down keys to go through the found entries (because I read the entry preview). Now, this is not possible. How can I achieve my old behavior with the current solution? Would the mulitple-ESC-thingy a way to go for the others, too?

@koppor
Copy link
Member

koppor commented Dec 5, 2015

Idea to solve my issue: ENTER keeps the search as is and just focuses the main table. (Only works if the auto completion is not active)

@simonharrer
Copy link
Contributor

ENTER does not work with auto-complete. I propose to keep it as it is currently implemented.

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

6 participants