„Zwischen der Auswahl und dem Text ist kaum Platz" - und das stimmte überall, nicht nur an der einen Stelle. Der Grund war messbar: der Abstand zur Beschriftung betrug 8 px, die Polsterung innerhalb des Feldes 11,2 px. Das Wort stand also näher am Rahmen als der Inhalt des Rahmens an demselben Rahmen, und genau daran erkennt das Auge einen zu engen Satz. Die Regel heisst jetzt: der Abstand nach aussen ist grösser als die Polsterung nach innen. Sie steht einmal in globals.css als .field-label, .field-row und .field-hint - vorher war sie ein Dutzend Mal als mt-1, mt-1.5 oder gap-2 an den Aufrufstellen verteilt, mit drei verschiedenen Werten, von denen keiner reichte. 33 Stellen umgestellt. Der Test prüft die Regel und nicht die Zahl: er liest die Polsterung von .field aus dem CSS und verlangt, dass beide Abstände darüber liegen. Gegengeprobt mit einem zu kleinen Wert, dann schlägt er an. Nachgemessen im Browser mit dem gebauten CSS: 8 px senkrecht gegen 5,6 px Innenpolsterung, 13,6 px waagerecht gegen 11,2 px.
162 lines
4.9 KiB
TypeScript
162 lines
4.9 KiB
TypeScript
'use client'
|
|
|
|
import { useActionState, useState } from 'react'
|
|
import { departmentAdminText } from '@/lib/i18n/departments'
|
|
import {
|
|
createDepartmentAction,
|
|
updateDepartmentAction,
|
|
type ActionResult,
|
|
} from './actions'
|
|
|
|
type Existing = {
|
|
id: string
|
|
key: string
|
|
name: string
|
|
description: string | null
|
|
minClearance: number
|
|
isArchived: boolean
|
|
memberCount: number | null
|
|
}
|
|
|
|
/**
|
|
* Anlegen und Ändern in einem Formular.
|
|
*
|
|
* Das Kürzel steht nach dem Anlegen fest: es steckt in der Adresse jeder
|
|
* Abteilungsseite, und ein wandernder Bezeichner macht geteilte Links wertlos.
|
|
* Wer sich vertan hat, legt neu an und archiviert die alte — archivieren
|
|
* statt löschen, weil an einer Abteilung Vermerke und Zuordnungen hängen.
|
|
*/
|
|
export function DepartmentForm({
|
|
levels,
|
|
existing,
|
|
}: {
|
|
levels: { level: number; name: string }[]
|
|
existing?: Existing
|
|
}) {
|
|
const [state, action, pending] = useActionState<ActionResult | null, FormData>(
|
|
existing ? updateDepartmentAction : createDepartmentAction,
|
|
null,
|
|
)
|
|
const [offen, setOffen] = useState(!existing)
|
|
|
|
if (existing && !offen) {
|
|
return (
|
|
<div className="flex flex-wrap items-baseline gap-x-4 gap-y-1">
|
|
<span className="font-mono text-xs text-text-dim">{existing.key}</span>
|
|
<span className="grow">{existing.name}</span>
|
|
{existing.memberCount === null ? null : (
|
|
<span className="font-condensed text-2xs text-text-dim">
|
|
{departmentAdminText.members} {existing.memberCount}
|
|
</span>
|
|
)}
|
|
{existing.isArchived ? (
|
|
<span className="font-condensed text-2xs text-text-dim">
|
|
{departmentAdminText.archived}
|
|
</span>
|
|
) : null}
|
|
<button type="button" onClick={() => setOffen(true)} className="btn btn-sm">
|
|
{departmentAdminText.edit}
|
|
</button>
|
|
</div>
|
|
)
|
|
}
|
|
|
|
return (
|
|
<form action={action} className="max-w-(--container-measure) space-y-4">
|
|
{existing ? <input type="hidden" name="id" value={existing.id} /> : null}
|
|
|
|
<div className="grid gap-4 sm:grid-cols-2">
|
|
<label className="block">
|
|
<span className="field-label">
|
|
{departmentAdminText.key}
|
|
</span>
|
|
{existing ? (
|
|
<p className="mt-1.5 font-mono text-sm">{existing.key}</p>
|
|
) : (
|
|
<>
|
|
<input name="key" required maxLength={21} className="field w-full font-mono" />
|
|
<span className="field-hint">
|
|
{departmentAdminText.keyHint}
|
|
</span>
|
|
</>
|
|
)}
|
|
</label>
|
|
|
|
<label className="block">
|
|
<span className="field-label">
|
|
{departmentAdminText.name}
|
|
</span>
|
|
<input
|
|
name="name"
|
|
required
|
|
maxLength={120}
|
|
defaultValue={existing?.name}
|
|
className="field w-full"
|
|
/>
|
|
</label>
|
|
</div>
|
|
|
|
<label className="block">
|
|
<span className="field-label">
|
|
{departmentAdminText.description}
|
|
</span>
|
|
<textarea
|
|
name="description"
|
|
rows={2}
|
|
maxLength={500}
|
|
defaultValue={existing?.description ?? ''}
|
|
className="field w-full"
|
|
/>
|
|
</label>
|
|
|
|
<div className="flex flex-wrap items-end gap-6">
|
|
<label className="block">
|
|
<span className="field-label">
|
|
{departmentAdminText.minClearance}
|
|
</span>
|
|
<select
|
|
name="minClearance"
|
|
defaultValue={existing?.minClearance ?? 0}
|
|
className="field"
|
|
>
|
|
{levels.map((level) => (
|
|
<option key={level.level} value={level.level}>
|
|
{level.level} · {level.name}
|
|
</option>
|
|
))}
|
|
</select>
|
|
<span className="field-hint">
|
|
{departmentAdminText.minClearanceHint}
|
|
</span>
|
|
</label>
|
|
|
|
{existing ? (
|
|
<label className="flex items-center gap-2 pb-2 text-sm">
|
|
<input type="checkbox" name="isArchived" defaultChecked={existing.isArchived} />
|
|
{departmentAdminText.archived}
|
|
</label>
|
|
) : null}
|
|
</div>
|
|
|
|
<div className="flex flex-wrap items-center gap-4">
|
|
<button type="submit" disabled={pending} aria-busy={pending} className="btn btn-primary">
|
|
{existing ? departmentAdminText.save : departmentAdminText.create}
|
|
</button>
|
|
{existing ? (
|
|
<button type="button" onClick={() => setOffen(false)} className="btn btn-sm">
|
|
{departmentAdminText.cancel}
|
|
</button>
|
|
) : null}
|
|
{state?.ok ? (
|
|
<span className="font-condensed text-2xs text-text-dim">
|
|
{existing ? departmentAdminText.saved : departmentAdminText.created}
|
|
</span>
|
|
) : null}
|
|
{state && !state.ok ? (
|
|
<span className="font-condensed text-2xs text-stamp-4">{state.error}</span>
|
|
) : null}
|
|
</div>
|
|
</form>
|
|
)
|
|
}
|