Opened 43 hours ago
Last modified 23 hours ago
#36829 assigned Bug
Breaking change in ClearableFileInput — at Version 1
| Reported by: | Johannes Maron | Owned by: | |
|---|---|---|---|
| Component: | Forms | Version: | 6.0 |
| Severity: | Release blocker | Keywords: | |
| Cc: | Johannes Maron, Antoliny | Triage Stage: | Unreviewed |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | yes |
Description (last modified by )
Howdy y'all,
#35892 flipped django.forms.widgets.ClearableFileInput.use_fieldset from false to true.
The change can be found here: https://github.com/django/django/blame/4187da258fe212d494cb578a0bc2b52c4979ab95/django/forms/widgets.py#L533
This changes the form rendering outside the admin too. Probably just a small oversight and not an intentional change, since changes weren't accompanied by release notes.
Anyhow, both the input and the use_fieldset are part of the public documentation, and changes must undergo deprecation.
The problem gets amplified with #36828, since the current fieldset implementation violates HTML and WCAG 2.1
The only affected version is 6.0. I would recommend providing a regression patch in the next bugfix release.
Cheers!
Joe