Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#13443 closed (fixed)

No required field indication in TabularInline

Reported by: Dirk Eschler Owned by: nobody
Component: contrib.admin Version:
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

In StackedInline the label of a required field is assigned the class attribute "required". In TabularInline (and GenericTabularInline) however there is no such indication. You have to save in order to know what fields are required. Usability wise it would be nice if the table head cell in a tabular inline also received the "required" class.

Change History (3)

comment:1 by Russell Keith-Magee, 14 years ago

milestone: → 1.2
Triage Stage: Unreviewed → Accepted

The 'required' field was added in the 1.2 cycle (or rather, restored in the 1.2 cycle - it existed in 0.96, and was lost in the newforms refactor). It exists on StackedInline by accident (due to a shared template fragment) - it should be added to TabularInline for consistency.

comment:2 by Russell Keith-Magee, 14 years ago

Resolution: → fixed
Status: new → closed

(In [13058]) Fixed #13443 -- Added required CSS class to the column headings on admin TabularInline formsets. Thanks to deschler for the report.

comment:3 by Jacob, 13 years ago

milestone: 1.2

Milestone 1.2 deleted

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