[bisq-network/bisq] Add missing filtering on lists (PR #5963)

chimp1984 notifications at github.com
Thu Jan 20 18:26:42 CET 2022


I have not looked closedly to the other filter predicates. But basically it shoud always take the display string as that is what users would expect to be able to filter. Maybe there are some cases where additional domain data filtering is useful as well, but should be additional to the display string case.
As we have not plenty of filters in the app it would be good to have them re-usable also with reusable predicates where it makes sense.

I am not so 100% sure if we should show the filter fields by default or extend it just when the user demand it. E.g. add a filter icon and when the user opens it show the field. That would help to not overload more the already overloaded UI.
We can persist the selection of hte icon, so users who prefer to have the filter get it automatically displayed at next restart.

Beside that great to see that feature added. 

-- 
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/5963#issuecomment-1017743827
You are receiving this because you are subscribed to this thread.

Message ID: <bisq-network/bisq/pull/5963/c1017743827 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20220120/e826f029/attachment-0001.htm>


More information about the bisq-github mailing list