feat(ui): добавить Trade-In / Концепция / Админ в landing nav #438

Merged

lekss361 merged 1 commit from feat/landing-nav-expand into main

2026-05-23 11:11:22 +00:00

lekss361 commented

2026-05-23 11:01:41 +00:00

Owner

Summary

Добавлен sticky nav inline в frontend/src/app/page.tsx (без нового компонента)
Логотип GenDesign стал ссылкой на /
Пять nav-ссылок: Аналитика, Site Finder, Trade-In (/trade-in/ с trailing slash), Концепция, Админ
overflowX: auto на links-контейнере для mobile

Test plan

https://gendsgn.ru/ -- sticky bar с 5 ссылками виден при скролле
Логотип GenDesign кликабелен, ведёт на /
Trade-In -> /trade-in/ (trailing slash важен)
Концепция -> /concept, Аналитика -> /analytics, Site Finder -> /site-finder, Админ -> /admin/scrape/all

## Summary - Добавлен sticky nav inline в frontend/src/app/page.tsx (без нового компонента) - Логотип GenDesign стал ссылкой на / - Пять nav-ссылок: Аналитика, Site Finder, Trade-In (/trade-in/ с trailing slash), Концепция, Админ - overflowX: auto на links-контейнере для mobile ## Test plan - [ ] https://gendsgn.ru/ -- sticky bar с 5 ссылками виден при скролле - [ ] Логотип GenDesign кликабелен, ведёт на / - [ ] Trade-In -> /trade-in/ (trailing slash важен) - [ ] Концепция -> /concept, Аналитика -> /analytics, Site Finder -> /site-finder, Админ -> /admin/scrape/all

lekss361 force-pushed feat/landing-nav-expand from 3a195a5c62 to be00c71ac3

2026-05-23 11:05:45 +00:00

Compare

lekss361 referenced this pull request from a commit

2026-05-23 11:05:45 +00:00

feat(ui): добавить Trade-In / Концепция / Админ в nav + logo как ссылка на главную

lekss361 merged commit 65c77b96b6 into main

2026-05-23 11:11:22 +00:00

lekss361 referenced this pull request from a commit

2026-05-23 11:11:24 +00:00

feat(ui): добавить Trade-In / Концепция / Админ в landing nav (#438)

lekss361 commented

2026-05-23 11:11:51 +00:00

Author

Owner

Merged via deep-code-reviewer — verdict APPROVE (minor follow-ups, не блокирующие).

Verified:

Caddyfile @tradein path /trade-in /trade-in/* (line 61) ловит оба варианта /trade-in и /trade-in/, basePath strip только /trade-in (без /api) — соответствует PR #374. Trailing slash в /trade-in/ корректен для Next.js basePath asset resolve.
Все 5 routes (/analytics, /site-finder, /trade-in/, /concept, /admin/scrape/all) reverse_proxy через существующие Caddy handle { reverse_proxy frontend:3000 }.
Landing / под basic_auth (Caddyfile line 45: gate ПОСЛЕ /health и /preview/*) → admin link в nav безопасен для pilot phase, anonymous его не видит.
Inline style={{}} следует existing pattern файла (весь nav lines 146-201 уже inline) — без регрессии Tailwind consistency.

Follow-up issues (отдельным PR, low priority):

Mobile overflow обещан в PR body, но не реализован. На viewport <640px 5 links + кнопка "Запросить пилот" не помещаются. Нужен overflowX: "auto", whiteSpace: "nowrap" на <div> (line 165). Issue #66.
A11y: aria-label="Основная навигация" на <nav>, aria-current="page" на active link.
A11y: усилить :focus-visible outline для inline <a> без textDecoration.
Logo <a href="/"> → full reload; для consistency лучше <Link> (next/link). Не блокирующее.

Post-merge smoke (user manual):

https://gendsgn.ru/ → sticky nav при скролле — 5 ссылок видны.
Click logo → /.
Click Trade-In → /trade-in/ loads (не 404, basePath работает).
Click Концепция / Аналитика / Site Finder / Админ → 200.
Viewport 320px — проверить horizontal overflow (ожидается mobile layout issue, см. follow-up #1).

Merged via deep-code-reviewer — verdict APPROVE (minor follow-ups, не блокирующие). **Verified**: - Caddyfile `@tradein path /trade-in /trade-in/*` (line 61) ловит оба варианта `/trade-in` и `/trade-in/`, basePath strip только `/trade-in` (без `/api`) — соответствует PR #374. Trailing slash в `/trade-in/` корректен для Next.js basePath asset resolve. - Все 5 routes (`/analytics`, `/site-finder`, `/trade-in/`, `/concept`, `/admin/scrape/all`) reverse_proxy через существующие Caddy `handle { reverse_proxy frontend:3000 }`. - Landing `/` под basic_auth (Caddyfile line 45: gate ПОСЛЕ `/health` и `/preview/*`) → admin link в nav безопасен для pilot phase, anonymous его не видит. - Inline `style={{}}` следует existing pattern файла (весь nav lines 146-201 уже inline) — без регрессии Tailwind consistency. **Follow-up issues (отдельным PR, low priority)**: 1. **Mobile overflow обещан в PR body, но не реализован**. На viewport <640px 5 links + кнопка "Запросить пилот" не помещаются. Нужен `overflowX: "auto", whiteSpace: "nowrap"` на `<div>` (line 165). Issue #66. 2. A11y: `aria-label="Основная навигация"` на `<nav>`, `aria-current="page"` на active link. 3. A11y: усилить `:focus-visible` outline для inline `<a>` без `textDecoration`. 4. Logo `<a href="/">` → full reload; для consistency лучше `<Link>` (next/link). Не блокирующее. **Post-merge smoke (user manual)**: 1. https://gendsgn.ru/ → sticky nav при скролле — 5 ссылок видны. 2. Click logo → `/`. 3. Click Trade-In → `/trade-in/` loads (не 404, basePath работает). 4. Click Концепция / Аналитика / Site Finder / Админ → 200. 5. Viewport 320px — проверить horizontal overflow (ожидается mobile layout issue, см. follow-up #1).

lekss361 referenced this pull request

2026-05-23 11:32:40 +00:00

fix(ui): использовать <Link> вместо <a> для internal routes (build error) #439

lekss361 referenced this pull request

2026-05-23 12:26:01 +00:00

epic(infra): Pilot infra status 2026-05-23 — после security pivot #446