mirror of
https://codeberg.org/PostERG/xamxam.git
synced 2026-05-07 03:29:19 +02:00
c2eff75789
Add App::flashAutofocus(fieldName) and consumeAutofocus() to the thin App helper so action handlers can identify which field caused a validation error and the form page can move browser focus directly to it on reload. Changes: - src/App.php — flashAutofocus() stores field name in _flash_autofocus session key; consumeAutofocus() drains it and returns the name (or null) - actions/formulaire.php — catch block maps exception messages to field names (auteurice, titre, synopsis, année, orientation, ap, finality, languages, tag, lien) and calls App::flashAutofocus() - actions/edit.php — catch block maps common edit errors to field names and calls App::flashAutofocus() - add.php — consumes the hint via App::consumeAutofocus() into $autofocusField; withAutofocus() helper merges autofocus=>true into $attrs for every field include; synopsis textarea gets inline autofocus - edit.php — same pattern with inline ternary merges and textarea autofocus - templates/partials/form/text-field.php — $attrs loop now emits bare attribute names (no ="...") when value === true, supporting autofocus, disabled, readonly etc. without special-casing - templates/partials/form/select-field.php — same boolean-attr support added; $attrs variable initialised to [] when caller omits it Closes WCAG 3.3.1 autofocus item in todo/04-accessibility.md.
61 lines
2.4 KiB
PHP
61 lines
2.4 KiB
PHP
<?php
|
|
/**
|
|
* Text field partial — single-line text / number / url / email input.
|
|
*
|
|
* Variables consumed:
|
|
* string $name — input name attribute (also used for id)
|
|
* string $label — visible label text
|
|
* string $value — current value (already htmlspecialchars'd by caller, or raw)
|
|
* string $type — input type; default 'text'
|
|
* bool $required — whether the field is required; default false
|
|
* string $placeholder — placeholder text; default ''
|
|
* string|null $hint — optional hint shown in <small> below the input
|
|
* string|null $id — override the id attribute (defaults to $name)
|
|
* array $attrs — extra HTML attributes as key=>value pairs (e.g. min/max for number)
|
|
*
|
|
* The partial does NOT call htmlspecialchars on $value — the caller is responsible.
|
|
*/
|
|
|
|
$type = $type ?? 'text';
|
|
$required = $required ?? false;
|
|
$placeholder = $placeholder ?? '';
|
|
$hint = $hint ?? null;
|
|
$id = $id ?? $name;
|
|
$attrs = $attrs ?? [];
|
|
|
|
$attrStr = '';
|
|
foreach ($attrs as $k => $v) {
|
|
if ($v === true) {
|
|
$attrStr .= ' ' . htmlspecialchars($k);
|
|
} else {
|
|
$attrStr .= ' ' . htmlspecialchars($k) . '="' . htmlspecialchars((string)$v) . '"';
|
|
}
|
|
}
|
|
?>
|
|
<div>
|
|
<label for="<?= htmlspecialchars($id) ?>"><?= htmlspecialchars($label) ?></label>
|
|
<?php if ($hint): ?>
|
|
<div>
|
|
<input type="<?= htmlspecialchars($type) ?>"
|
|
id="<?= htmlspecialchars($id) ?>"
|
|
name="<?= htmlspecialchars($name) ?>"
|
|
value="<?= $value ?>"
|
|
<?= $required ? 'required' : '' ?>
|
|
<?= $placeholder ? 'placeholder="' . htmlspecialchars($placeholder) . '"' : '' ?>
|
|
<?= $attrStr ?>>
|
|
<small><?= htmlspecialchars($hint) ?></small>
|
|
</div>
|
|
<?php else: ?>
|
|
<input type="<?= htmlspecialchars($type) ?>"
|
|
id="<?= htmlspecialchars($id) ?>"
|
|
name="<?= htmlspecialchars($name) ?>"
|
|
value="<?= $value ?>"
|
|
<?= $required ? 'required' : '' ?>
|
|
<?= $placeholder ? 'placeholder="' . htmlspecialchars($placeholder) . '"' : '' ?>
|
|
<?= $attrStr ?>>
|
|
<?php endif; ?>
|
|
</div>
|
|
<?php
|
|
// Reset consumed variables so includes in a loop don't bleed state.
|
|
unset($type, $required, $placeholder, $hint, $id, $attrs, $attrStr, $k, $v);
|