Opened 18 years ago
Closed 18 years ago
#6606 closed (invalid)
django.newforms.form.BoundField.as_widget doesn't merge widget.attrs
| Reported by: | moep | Owned by: | moep |
|---|---|---|---|
| Component: | Forms | Version: | dev |
| Severity: | Keywords: | ||
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
django.newforms.form.BoundField.as_widget doesn't merge widget.attrs
wich means if you specify additional attr in Field.widget_attrs they never make it into Widget.render
I could write a patch but I'm not quite sure if as_widget is the right place to do it. The return value
of widget.attrs gets set to Field.widget_attrs so widget.render could do the merging as well, but then
you would have to change almost every widget.render function.
Change History (2)
comment:1 by , 18 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:2 by , 18 years ago
| Resolution: | → invalid |
|---|---|
| Status: | assigned → closed |
Note:
See TracTickets
for help on using tickets.
merging is done Widget.build_attrs ... i might have been a bit to overhasty :)