Opened 11 years ago

Closed 11 years ago

#19874 closed Cleanup/optimization (fixed)

Add id to <ul> element in CheckboxSelectMultiple

Reported by: l3on Owned by: Baptiste Mispelon
Component: Forms Version: dev
Severity: Normal Keywords:
Cc: bmispelon@… Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Please,

consider to add an ID to <ul> element in CheckboxSelectMultiple.

The patch attached fixes the problem (it's based on git/develop branch).

Thanks in advance!

Leo.

Attachments (1)

widgets-add_id_to_CheckboxSelectMultiple.patch (580 bytes ) - added by l3on 11 years ago.
add id to CheckboxSelectMultiple

Download all attachments as: .zip

Change History (7)

by l3on, 11 years ago

add id to CheckboxSelectMultiple

comment:1 by Tim Graham, 11 years ago

Component: DocumentationForms

comment:2 by Carl Meyer, 11 years ago

Triage Stage: UnreviewedAccepted
Type: UncategorizedCleanup/optimization

Seems reasonable to me; the id is currently only used to construct per-option numbered IDs, it makes sense that the base ID would be applied to the container.

comment:3 by Tomek Paczkowski, 11 years ago

Needs documentation: set
Needs tests: set

comment:4 by Baptiste Mispelon, 11 years ago

Cc: bmispelon@… added
Has patch: unset
Needs documentation: unset
Needs tests: unset
Owner: changed from nobody to Baptiste Mispelon
Status: newassigned
Version: master

I upgraded the patch to use format_html instead of string interpolation.

I also fixed the test suite and added some documentation of the feature.

The pull request can be found there: https://github.com/django/django/pull/999

comment:5 by Claude Paroz, 11 years ago

Has patch: set
Triage Stage: AcceptedReady for checkin

comment:6 by Claude Paroz <claude@…>, 11 years ago

Resolution: fixed
Status: assignedclosed

In 844fbc85c1d419ed7ecbb7208bd392c1770ea72d:

Fixed #19874: Apply id attribute to the outer <ul> of CheckboxSelectMultiple

Note: See TracTickets for help on using tickets.
Back to Top