Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7k
HTML rendering for filters#3315
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
jpadilla commentedAug 21, 2015
@tomchristie looking great, any specific reason for introducing the requirement for crispy forms? |
lovelydinosaur commentedAug 21, 2015
@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 |
carltongibson commentedSep 1, 2015
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... |
lovelydinosaur commentedSep 2, 2015
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. |
carltongibson commentedSep 2, 2015
OK. Fine. I'll put something together. |
Conflicts:rest_framework/compat.pyrest_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...