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
Improvement ideas for the search result filters #1400
Comments
I think this is needlessly complex, and tries to solve something that shouldn't be an issue. If there are any proven concerns about memory usage and performance, these should be addressed instead (this is something I've focused on improving, and could do more work on it if necessary). You're essentially shooting yourself in the foot by going as low as 250. |
That's fair. Maybe we shouldn't be allowed to do so if there is no real upside? Unless there's anything you see fit to implement from what was suggested, this issue can be closed now. |
I've previously thought about removing the "Maximum total results per search" option, since it doesn't serve a large purpose anymore. This can stay open, I'll take a look at the other suggestions at some point. :) |
Thank you @redactedscribe indeed your ideas have a valid basis... and yes this has also inspired @mathiascode and myself to look into what exactly causes these types of UI obstructions when using the Result Filters features, as well as some other possible solutions you might be interested in, see issues #1498 #1499 #1500 DONE
DONE #1693 In addition to the above mentioned recent improvements, heres my suggestion that could potentially provide the final piece of requested functionality in the OP... "I miss being able to turn off the Result Filters, not just hide them, as used to be possible via the main UI. Maybe the "Results Filters" button could be given a Ctrl + click [or some other] feature to toggle their on/off state, instead of the default plain click which currently just toggles their visibility." TODO: TODO: Proposed compromise to this in PR #2404
If the text entry boxes were still populated with the most recent entries when it is activated again, then this would close #1400 Note: The appearance of the icon used on Result Filters button itself is discussed in #1601 |
Is this still something you want? |
The OP's suggestion to synchronize the enabling/disabling of the result filters along with the toolbar visibility is a good one, yes.
Indeed it would be ideal if |
Thanks for the improvements! |
Describe the solution you'd like
Currently, it's possible to have a "Maximum results per search visible at a time" smaller than "Maximum total results per search". I haven't taken the time to prove it, but I like to keep the former smaller than the latter to help keep Nicotine+'s memory footprint reasonable and its performance from degrading.
Assuming: "Maximum results per search visible at a time":
250
, and "Maximum total results per search"1000
.I have some usability improvement ideas for when configured as above:
Other ideas:
I miss being able to turn off the Result Filters, not just hide them, as used to be possible via the main UI. Maybe the "Results Filters" button could be given a Ctrl + click feature to toggle their on/off state, instead of the default plain click which currently just toggles their visibility.
A quick way to clear an individual search result filter field without clearing them all could be nice. E.g. the typical "X" to the right of a field. If not done well, these Xs to clear can sometimes be quite annoying to target and click intentionally/successfully.
Hopefully those make sense and can at minimum serve as some inspiration.
Thanks!
The text was updated successfully, but these errors were encountered: