Opened 5 years ago

Closed 5 years ago

Last modified 3 years ago

#13443 closed (fixed)

No required field indication in TabularInline

Reported by: deschler 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 5 years ago by russellm

  • milestone set to 1.2
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to 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 Changed 5 years ago by russellm

  • Resolution set to fixed
  • Status changed from new to 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 Changed 3 years ago by jacob

  • milestone 1.2 deleted

Milestone 1.2 deleted

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