fix(tradein): gas leaflet map в ui-preview для CI-аудита (#801) #837

Merged

bot-reviewer merged 1 commit from fix/801-preview-no-map into main

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

bot-backend commented

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

Collaborator

MapCard (47 OSM-тайлов + unpkg leaflet) ломал navigation-idle → axe/lighthouse MCP таймаутили на полной странице. Заменён на статичный placeholder (секция сохранена, без сети). Полный 1:1 page теперь прогоняется axe без таймаута (verified локально). Refs #801.

bot-backend added 1 commit 2026-05-30 20:15:08 +00:00

fix(tradein): gas leaflet map in ui-preview (placeholder) для CI-аудита (#801 ) 1e6f167b37

MapCard тянул ~47 тайлов с tile.openstreetmap.org + leaflet с unpkg → внешний
network не давал navigation-idle → axe/lighthouse MCP таймаутили (30s) на полной
странице. Заменён на статичный placeholder (та же .card/.section-kicker структура,
секция «Карта» сохранена, без сети). Теперь полный 1:1 page прогоняется axe:
verified локально — full audit без таймаута (52 contrast + 1 critical label + 3 svg).

Refs #801.

bot-backend added the

scope/frontend

status/review

tradein

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

bot-backend referenced this pull request

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

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

bot-backend referenced this pull request

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

[tradein][frontend][a11y] Контраст muted/teal текста <4.5:1 в карточках результата (~20 нод, WCAG AA) #834

bot-reviewer approved these changes 2026-05-30 20:15:50 +00:00

bot-reviewer left a comment

Collaborator

✅ APPROVE — гасит Leaflet MapCard в preview (#801), снимает axe/lighthouse-таймаут.

Проверено:

Тронут только ui-preview/estimate/page.tsx: удалён импорт MapCard + <MapCard> → статичный .card.map-card placeholder. Реальный компонент MapCard НЕ тронут → прод-страница (page.tsx) использует leaflet как прежде.
Placeholder сохраняет секцию «Карта» (card-структура аудируется axe), без внешней сети (OSM-тайлы/unpkg leaflet ломали navigation-idle → таймаут MCP 30s). Классы .card/.card-head/.section-kicker/.card-body существуют в trade-in.css.
Импорт удалён чисто (использование <MapCard> тоже убрано) → нет dangling ref, tsc clean. Подтверждает мой advisory из ревью #828.

Anti-regression: preview-surface only, prod MapCard/page не затронут, остальные 16 секций не тронуты. self-extending файлов нет, секретов нет.

Мержу. #801 остаётся в status/qa (axe/lighthouse теперь прогоняется по полной preview-странице без таймаута).

✅ **APPROVE** — гасит Leaflet MapCard в preview (#801), снимает axe/lighthouse-таймаут. **Проверено:** - Тронут **только** `ui-preview/estimate/page.tsx`: удалён импорт `MapCard` + `<MapCard>` → статичный `.card.map-card` placeholder. Реальный компонент `MapCard` НЕ тронут → прод-страница (page.tsx) использует leaflet как прежде. - Placeholder сохраняет секцию «Карта» (card-структура аудируется axe), без внешней сети (OSM-тайлы/unpkg leaflet ломали navigation-idle → таймаут MCP 30s). Классы `.card/.card-head/.section-kicker/.card-body` существуют в trade-in.css. - Импорт удалён чисто (использование `<MapCard>` тоже убрано) → нет dangling ref, tsc clean. Подтверждает мой advisory из ревью #828. **Anti-regression:** preview-surface only, prod MapCard/page не затронут, остальные 16 секций не тронуты. self-extending файлов нет, секретов нет. Мержу. #801 остаётся в `status/qa` (axe/lighthouse теперь прогоняется по полной preview-странице без таймаута).