-
Notifications
You must be signed in to change notification settings - Fork 5
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
Allow filtering from columns #144
base: main
Are you sure you want to change the base?
Conversation
We should probably figure out how to do the bulk-selection first so the forms don't interfere with each other. I assume the form will have hidden fields for the other values so we don't lose the existing GET-args? |
The icon needs be flatter =) |
hmm perhaps, but I think the bulk selection will be a completely separate flow from the filtering, no? or is it the mass checkbox that you're concerned about?
I was thinking to just cross
I went with something that is in unicode. I could not find anything that worked better 😅. Maybe we need to think about adopting an icon pack (fontawesome?) |
Yep. |
As for supporting other types than text-input, we'd have to extend the column-objects. |
Haven't looked at code yet but in general multiple |
51139cc
to
b39d9b8
Compare
Allow specifiying when a column may be "filterable" (by specifying the
IncidentTableColumn.filter_field
). This means:filter_field
as the query param to filter incidentsTODO:
IncidentTableColumn.filter_field
is setLimitations: