Opened 6 years ago

Closed 6 years ago

Last modified 5 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: UI/UX:

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 Changed 6 years ago by Russell Keith-Magee

milestone: 1.2
Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedAccepted

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 Changed 6 years ago by Russell Keith-Magee

Resolution: fixed
Status: newclosed

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

comment:3 Changed 5 years ago by Jacob

milestone: 1.2

Milestone 1.2 deleted

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