-
-
Notifications
You must be signed in to change notification settings - Fork 732
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed Grouped Choices for Radios and Checkboxes #1155
Conversation
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.
A few notes to look at. The pre-work here is proving useful as we can clearly see what the changes are with quite a significant change to the underlying template.
...ests/results/bootstrap4/test_layout/test_use_custom_control_is_used_in_checkboxes_false.html
Outdated
Show resolved
Hide resolved
...ests/results/bootstrap4/test_layout/test_use_custom_control_is_used_in_checkboxes_false.html
Outdated
Show resolved
Hide resolved
...rms/tests/results/bootstrap4/test_layout/test_use_custom_control_is_used_in_radio_false.html
Show resolved
Hide resolved
97b91a7
to
d937f5d
Compare
Hi @smithdc1 could you please re-examine this PR. It seems like it's been the cause of quite a number of issues raised for django-crispy-forms. I think we could probably find a way to re-do this that doesn't necessarily break a lot of the existing implementations. I ask because you're a lot more familiar with the code base. All the same, I want to really thank you for everything you've done to make this possible. |
So here we are 馃帀 This fixes the long standing bug with grouped choices sets.
Fixes #690 and Follows #1119 and #813
I'll leave this here for comments, and look at pulling some of the related changes into separate PRs. There's also a few things which I want to take a closer look at.
required
attribute ontest_use_custom_control_is_used_in_radio_true.html