feature: add per column searching to tables#386
Conversation
|
WIP to get a quick review/sanity check on this before I do it for the rest of the tables, only done for the devices table right now. Also considering removing the regular search bar in favor of these, only downside is that its a bit tougher to use on mobile. |
b-rowan
force-pushed
the
table-search-updates
branch
from
528b424 to
3a69405
Compare
b-rowan
force-pushed
the
table-search-updates
branch
from
d267f26 to
c0fe746
Compare
This is to prevent conflicts when POSTing from the datatables to request specific searches. POST should now be used for requesting information, such as for the datatables or login credentials, while PUT should now be used for resource creation.
b-rowan
force-pushed
the
table-search-updates
branch
from
c0fe746 to
599bf7d
Compare
|
Sorry for the late feedback, I am still on vacation. Unfortunately, I must say, I am not a fan. I find the tables look quite ugly now. More importantly, as you noted correctly, on mobile the new search fields are quite unusable. I would rather look into extending the main search field to support tags like e.g. GitHub's search does. So, e.g. to search only the versions one would have to specify Of course, I have no idea how hard that is, but, I believe it would drastically improve the user experience. |
3 participants
Adds per column searching to the tables, allowing for much better filtering.