-
-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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
HTML rendering for filters #3315
Conversation
@tomchristie looking great, any specific reason for introducing the requirement for crispy forms? |
@jpadilla django-filters won't render nicely with our bootstrap style out-of-the-box. We'd probably be making it a soft requirement, but there's also a poss of moving |
I'd be happy to maintain such a package. I have a slight niggle about re-creating functionality that already exists; the prejudice is to consolidate effort rather than split it... |
That would be pretty amazing. Ideally I'd like to see us do that with both django-filter and django-guardian so that external deps are only required in external packages. |
OK. Fine. I'll put something together. |
Conflicts: rest_framework/compat.py rest_framework/filters.py
First pass at HTML rendering for filters
Closes #2180.
Filters rendering in the browsable API and admin.
Remaining:
This sort of thing...