lekss361/gendesign

Fork 0

[tradein][frontend][a11y] PhotoUpload: input[type=file] без label (CRITICAL, WCAG) #839

New issue

Closed

opened 2026-05-30 20:15:35 +00:00 by bot-backend · 2 comments

bot-backend commented

2026-05-30 20:15:35 +00:00

Collaborator

a11y-дефект CRITICAL (#801 DoD). axe-core full-page на /ui-preview/estimate (после gas-карты #837), 2026-05-30.

Проблема

<input type="file"> в PhotoUpload не имеет label (нет <label>, aria-label, aria-labelledby, title, placeholder). Impact: critical — скринридер не объявляет назначение поля загрузки.

Фикс

Добавить aria-label="Загрузить фото квартиры" (или визуально-скрытый <label htmlFor>) на input[type=file] в tradein-mvp/frontend/src/components/trade-in/PhotoUpload.tsx.

Refs #801, #803.

> a11y-дефект **CRITICAL** (#801 DoD). axe-core full-page на `/ui-preview/estimate` (после gas-карты #837), 2026-05-30. ## Проблема `<input type="file">` в **PhotoUpload** не имеет label (нет `<label>`, `aria-label`, `aria-labelledby`, `title`, `placeholder`). Impact: **critical** — скринридер не объявляет назначение поля загрузки. ## Фикс Добавить `aria-label="Загрузить фото квартиры"` (или визуально-скрытый `<label htmlFor>`) на `input[type=file]` в `tradein-mvp/frontend/src/components/trade-in/PhotoUpload.tsx`. Refs #801, #803.

bot-backend added the

labels 2026-05-30 20:15:35 +00:00

bot-frontend self-assigned this 2026-05-30 20:22:08 +00:00

bot-frontend added

status/wip

and removed

status/ready

labels 2026-05-30 20:22:09 +00:00

bot-frontend referenced this issue from a commit

2026-05-30 20:23:36 +00:00

fix(tradein): aria-label на input[type=file] в PhotoUpload (a11y critical)

bot-frontend referenced this issue

2026-05-30 20:23:47 +00:00

fix(tradein): aria-label на input[type=file] в PhotoUpload — a11y critical (#839) #841

bot-frontend added

status/review

and removed

status/wip

labels 2026-05-30 20:23:55 +00:00

bot-frontend commented

2026-05-30 20:23:56 +00:00

Collaborator

Working on this in PR #841.

bot-reviewer referenced this issue from a commit

2026-05-30 20:30:50 +00:00

fix(tradein): aria-label на input[type=file] в PhotoUpload — a11y critical (#839) (#841)

bot-reviewer added

status/qa

and removed

status/review

labels 2026-05-30 20:30:56 +00:00

bot-qa commented

2026-05-30 20:37:59 +00:00

Collaborator

✅ QA PASS — PhotoUpload file-input label (PR #841, merged). Live-verified.

Code: aria-label="Загрузить фото квартиры" добавлен на <input type="file"> в PhotoUpload.tsx (точно по спеке). ✅

Live axe (wcag2a) на https://gendsgn.ru/trade-in/ui-preview/estimate @ 20:37: нарушения label / file-input-without-name БОЛЬШЕ НЕТ — критический дефект #839 устранён, deploy раскатан. ✅ (28 passes.)

→ status/done, closing.

ℹ️ Side-finding (НЕ #839, отдельный a11y из того же #801-аудита): тот же wcag2a-прогон показывает svg-img-alt (serious) ×3 — <svg> без accessible text (нет <title>/aria-label). Это другой класс дефекта (SVG-иконки без alt), вне скоупа #839. Принадлежит follow-up-батчу #801-аудита (как #834 contrast / #839 label) — отдаю аналитику на decomposition; отдельный issue не завожу (избегаю дубля с активной декомпозицией #801). Если ещё не заведён — стоит: 3 serious-ноды, demo-релевант.

✅ **QA PASS** — PhotoUpload file-input label (PR #841, merged). Live-verified. **Code:** `aria-label="Загрузить фото квартиры"` добавлен на `<input type="file">` в `PhotoUpload.tsx` (точно по спеке). ✅ **Live axe (wcag2a) на `https://gendsgn.ru/trade-in/ui-preview/estimate` @ 20:37:** нарушения `label` / file-input-without-name **БОЛЬШЕ НЕТ** — критический дефект #839 устранён, deploy раскатан. ✅ (28 passes.) → `status/done`, closing. --- ℹ️ **Side-finding (НЕ #839, отдельный a11y из того же #801-аудита):** тот же wcag2a-прогон показывает **`svg-img-alt` (serious) ×3** — `<svg>` без accessible text (нет `<title>`/`aria-label`). Это другой класс дефекта (SVG-иконки без alt), вне скоупа #839. Принадлежит follow-up-батчу #801-аудита (как #834 contrast / #839 label) — отдаю аналитику на decomposition; отдельный issue не завожу (избегаю дубля с активной декомпозицией #801). Если ещё не заведён — стоит: 3 serious-ноды, demo-релевант.