diff options
| author | Johannes Maron <johannes@maron.family> | 2025-12-27 12:28:26 +0100 |
|---|---|---|
| committer | Jacob Walls <jacobtylerwalls@gmail.com> | 2025-12-31 10:18:06 -0500 |
| commit | b7b5465b1c026dda7de646fca15ca1e97886d021 (patch) | |
| tree | e1c0da5fd91ee9921b0488790021e071b2d40408 /tests/forms_tests/widget_tests/test_clearablefileinput.py | |
| parent | 90daa655486c7fc69e9dad41c5e96f00339a3b9b (diff) | |
[6.0.x] Fixed #36829 -- Reverted value of ClearableFileInput.use_fieldset to True.
There was unresolved discussion regarding whether to set
ClearableFileInput.use_fieldset to True or False when use_fieldset was
introduced in Django 4.1, since the clear checkbox appears only
sometimes. Although using <fieldset> is likely desirable, since the
primary motivation in #35892 was just to improve markup in the admin,
and a deprecation path was not provided for general form usage, future
work is deferred to #36828.
Regression in 4187da258fe212d494cb578a0bc2b52c4979ab95.
Thanks Tim Graham, Antoliny, and David Smith for triage.
Diffstat (limited to 'tests/forms_tests/widget_tests/test_clearablefileinput.py')
| -rw-r--r-- | tests/forms_tests/widget_tests/test_clearablefileinput.py | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/tests/forms_tests/widget_tests/test_clearablefileinput.py b/tests/forms_tests/widget_tests/test_clearablefileinput.py index 8d3bff4b45..ae54cc4b5d 100644 --- a/tests/forms_tests/widget_tests/test_clearablefileinput.py +++ b/tests/forms_tests/widget_tests/test_clearablefileinput.py @@ -246,19 +246,18 @@ class ClearableFileInputTest(WidgetTest): ) form = TestForm() - self.assertIs(self.widget.use_fieldset, True) + self.assertIs(self.widget.use_fieldset, False) self.assertHTMLEqual( - '<div><fieldset><legend for="id_field">Field:</legend>' - '<input id="id_field" name="field" type="file" required></fieldset></div>' - '<div><fieldset><legend for="id_with_file">With file:</legend>Currently: ' + '<div><label for="id_field">Field:</label>' + '<input id="id_field" name="field" type="file" required></div>' + '<div><label for="id_with_file">With file:</label>Currently: ' '<a href="something">something</a><br>Change:<input type="file" ' - 'name="with_file" id="id_with_file"></fieldset></div>' - '<div><fieldset><legend for="id_clearable_file">Clearable file:</legend>' + 'name="with_file" id="id_with_file"></div>' + '<div><label for="id_clearable_file">Clearable file:</label>' 'Currently: <a href="something">something</a><input ' 'type="checkbox" name="clearable_file-clear" id="clearable_file-clear_id">' '<label for="clearable_file-clear_id">Clear</label><br>Change:' - '<input type="file" name="clearable_file" id="id_clearable_file">' - "</fieldset></div>", + '<input type="file" name="clearable_file" id="id_clearable_file"></div>', form.render(), ) |
