Opened 16 years ago
Closed 16 years ago
#8993 closed (wontfix)
Rendering formsets as tables with a row per form
Reported by: | Owned by: | nobody | |
---|---|---|---|
Component: | Forms | Version: | dev |
Severity: | Keywords: | formsets | |
Cc: | Triage Stage: | Unreviewed | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
In the default as_table method of formsets it states:
# XXX: there is no semantic division between forms here, there # probably should be. It might make sense to render each form as a # table row with each field as a td.
The attached patch provides an 'as_table_wide' method, which does this. It calls an 'as_row' method on each form. This 'as_row' method is also supplied in the patch.
It is also useful to be able to easily include extra columns in the table, that are not form entries, but which depend on the form. This is especially true for model formsets, where the contents of the extra columns depends on the same model instance as the form, eg for fields of the model that may not change, but which should be displayed. This can be done by providing a form class with its own 'as_row' method, which modifies the standard one.
Attachments (1)
Change History (3)
by , 16 years ago
Attachment: | forms_formsets.diff added |
---|
comment:1 by , 16 years ago
Component: | Uncategorized → Forms |
---|
comment:2 by , 16 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
I don't think we need to proliferate
as_*
methods; that's what templates are for.