Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#20567 closed Cleanup/optimization (fixed)

Document BoundField.id_for_label and how to set the 'id' of a widget

Reported by: littlepig Owned by: timo
Component: Documentation Version: 1.5
Severity: Normal Keywords:
Cc: littlepig, timograham@… Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

By the accepted answer on this stackoverflow question, it seems it is possible to override the 'id' attribute of a widget without interfering with django's forms framework.

I think this issue of initial values of attrs should be better documented on built-in widgets.

Change History (5)

comment:1 Changed 2 years ago by timo

  • Needs documentation unset
  • Needs tests unset
  • Owner changed from nobody to timo
  • Patch needs improvement unset
  • Status changed from new to assigned
  • Triage Stage changed from Unreviewed to Accepted

I'll work on this. Think it makes sense to also document BoundField.id_for_label as noted in #12856.

comment:2 Changed 2 years ago by timo

  • Cc timograham@… added
  • Has patch set
  • Summary changed from Documenting form widget initial attrs to Document BoundField.id_for_label and how to set the 'id' of a widget

comment:3 Changed 2 years ago by Kamu

  • Triage Stage changed from Accepted to Ready for checkin

comment:4 Changed 2 years ago by Tim Graham <timograham@…>

  • Resolution set to fixed
  • Status changed from assigned to closed

In 175a102ddceae2ebd1a4e03916a23adafc35bda5:

Fixed #20567 - Documented BoundField.id_for_label.

Thanks littlepig for the suggestion.

comment:5 Changed 2 years ago by Tim Graham <timograham@…>

In 3bd9852401cfbc8a1dfec4082303a5514ee69bf4:

[1.5.x] Fixed #20567 - Documented BoundField.id_for_label.

Thanks littlepig for the suggestion.

Backport of 175a102ddc from master.

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