New event: space out the stacked cards, rework the kind picker to 4 tiles
- The kind-tile grid, title/date/location card, Who card, and Answers-close card were direct children of the form with no gap between them at all -- the wrapping flex/gap-4 on coach-sheet's own content div only ever applied between the header bar and the form, not what's inside it. The form itself is now flex flex-col gap-4. - The picker offered Practice/Game/Other, none of which actually changed which fields render. Reworked to Practice/Game/Tournament/Meeting (2x2) -- the four kinds worth adding from the app; social/other stay desktop-only. Also narrowed the form's own kind choices to match server-side, not just cosmetically in the template. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
@@ -45,6 +45,11 @@ IN_STATUSES = [Attendance.AttendanceStatus.PRESENT, Attendance.AttendanceStatus.
|
||||
#: Distinct from NO_RESPONSE ("silent"), which is a non-answer rather than a no.
|
||||
OUT_STATUSES = [Attendance.AttendanceStatus.ABSENT, Attendance.AttendanceStatus.EXCUSED, Attendance.AttendanceStatus.NOT_SELECTED]
|
||||
|
||||
#: The event kinds CoachCreateEventView's tile picker offers -- social/other
|
||||
#: stay desktop-only (management.forms.EventForm keeps the full list), since
|
||||
#: neither has a tile here.
|
||||
COACH_EVENT_KINDS = [Event.EventKind.TRAINING, Event.EventKind.GAME, Event.EventKind.TOURNAMENT, Event.EventKind.MEETING]
|
||||
|
||||
#: How long an event stays "current" (CoachTodayView's session card, and the
|
||||
#: missing-line-up nudge) past the moment it starts -- events.start__gte=now
|
||||
#: alone would flip to the next session the instant this one begins, while
|
||||
@@ -321,6 +326,10 @@ class CoachCreateEventView(CoachScopeMixin, LoginRequiredMixin, TemplateView):
|
||||
# referee-count control this screen has no use for; construct_instance
|
||||
# skips deleted fields entirely, leaving the instance's own default.
|
||||
del form.fields["max_referees"]
|
||||
# Narrowed to the four kinds the tile picker actually offers -- social/
|
||||
# other don't get their own tile, and this keeps a tampered request from
|
||||
# setting one anyway (the desktop form still offers the full list).
|
||||
form.fields["kind"].choices = [choice for choice in form.fields["kind"].choices if choice[0] in COACH_EVENT_KINDS]
|
||||
# The desktop searchable multi-select relies on management's own JS
|
||||
# widget, not loaded here -- plain checkboxes work without it and
|
||||
# read better on a phone regardless.
|
||||
|
||||
Reference in New Issue
Block a user