-
Notifications
You must be signed in to change notification settings - Fork 39
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
Add custom basic form templates #3034
Add custom basic form templates #3034
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3034 +/- ##
==========================================
- Coverage 60.47% 60.45% -0.02%
==========================================
Files 605 605
Lines 43816 43818 +2
Branches 48 48
==========================================
- Hits 26497 26490 -7
- Misses 17307 17316 +9
Partials 12 12 ☔ View full report in Codecov by Sentry. |
5dc3632
to
70f3d83
Compare
02301b6
to
845ef63
Compare
548cf02
to
d54ecfe
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much better, but a couple of WTFs...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright, ivar
/cvar
is explained, nothing further to note here :)
This uses the form mechanisms that were introduced in #3034 to replace crispy-forms.
This uses the form mechanisms that were introduced in #3034 to replace crispy-forms.
This uses the form mechanisms that were introduced in #3034 to replace crispy-forms.
This uses the form mechanisms that were introduced in #3034 to replace crispy-forms.
This uses the form mechanisms that were introduced in #3034 to replace crispy-forms.
This uses the form mechanisms that were introduced in #3034 to replace crispy-forms.
Adds basic reusable form building blocks for #2794.
Note:
set_flat_form_attributes()
is meant to replace crispy FormHelper but keeping only relevant subset of attributes and logic. Just add more attributes toset_flat_form_attributes
if needed./custom_crispy_templates/_form_content.html
can be included in template directly (withoutflat_form.html
wrapper) when uncrispyfying forms that are initialised withself.helper.form_tag = False
Css classes and tag attributes in new templates are meant to work as minimally viable reimplementation of: