fix(sf-06): MarketTab reorder — Competitors → BestLayouts → Velocity первыми #272

Merged

lekss361 merged 1 commit from fix/sf-06-market-tab-reorder into main

2026-05-17 08:57:18 +00:00

lekss361 commented

2026-05-17 08:55:30 +00:00

Owner

Summary

Reorder JSX блоков в MarketTab для лучшего UX — самое важное сверху.

Старый порядок: Pipeline24mo → MarketTrend → Velocity → Competitors → SuccessRecommendation → BestLayouts

Новый порядок: Competitors → BestLayouts → Velocity → MarketTrend → Pipeline24mo → SuccessRecommendation

Изменения

frontend/src/components/site-finder/MarketTab.tsx — только JSX swap, без изменений логики/hooks/props

Test plan

После deploy /site-finder?tab=market — блоки в новом порядке (Competitors первый)
Regression — другие tabs (overview, land) не сломались

Related: epic #271 item #6 (Wave 1)

## Summary Reorder JSX блоков в `MarketTab` для лучшего UX — самое важное сверху. **Старый порядок:** `Pipeline24mo → MarketTrend → Velocity → Competitors → SuccessRecommendation → BestLayouts` **Новый порядок:** `Competitors → BestLayouts → Velocity → MarketTrend → Pipeline24mo → SuccessRecommendation` ## Изменения - `frontend/src/components/site-finder/MarketTab.tsx` — только JSX swap, без изменений логики/hooks/props ## Test plan - [ ] После deploy `/site-finder?tab=market` — блоки в новом порядке (Competitors первый) - [ ] Regression — другие tabs (`overview`, `land`) не сломались Related: epic #271 item #6 (Wave 1)

lekss361 added 1 commit 2026-05-17 08:55:30 +00:00

fix(sf-06): MarketTab reorder — Competitors → BestLayouts → Velocity первыми 81bf798c27

lekss361 merged commit 9127cf2060 into main

2026-05-17 08:57:18 +00:00

lekss361 referenced this pull request from a commit

2026-05-17 08:57:18 +00:00

Merge pull request 'fix(sf-06): MarketTab reorder — Competitors → BestLayouts → Velocity первыми' (#272) from fix/sf-06-market-tab-reorder into main

lekss361 commented

2026-05-17 08:57:34 +00:00

Author

Owner

Merged via deep-code-reviewer — verdict APPROVE.

Verification:

Чистый JSX swap: все 6 блоков переставлены, условия рендеринга (hasTrend, hasVelocity, hasRecommendation, data.competitors.length > 0, data.pipeline_24mo) сохранены
Props не тронуты (cadNum={data.cad_num}, trend={data.market_trend}, competitors, districtName — те же)
Hooks не задеты (изменения только внутри JSX return (...), hooks выше line 28)
EmptyState fallback !hasAny остался на месте
Стили inline и комментарии-якоря (D4 (#36), D2 (#34), Issue #113) переехали вместе с блоками
Симметричный +25/-25 = подтверждённый reorder

Smoke после деплоя:

/site-finder?tab=market → новый порядок: Competitors → BestLayouts → Velocity → MarketTrend → Pipeline24mo → SuccessRecommendation
Regression: tab=overview / tab=land без изменений

Related: epic #271 item #6 (Wave 1).

Merged via deep-code-reviewer — verdict APPROVE. **Verification:** - Чистый JSX swap: все 6 блоков переставлены, условия рендеринга (`hasTrend`, `hasVelocity`, `hasRecommendation`, `data.competitors.length > 0`, `data.pipeline_24mo`) сохранены - Props не тронуты (`cadNum={data.cad_num}`, `trend={data.market_trend}`, `competitors`, `districtName` — те же) - Hooks не задеты (изменения только внутри JSX `return (...)`, hooks выше line 28) - `EmptyState` fallback `!hasAny` остался на месте - Стили inline и комментарии-якоря (`D4 (#36)`, `D2 (#34)`, `Issue #113`) переехали вместе с блоками - Симметричный +25/-25 = подтверждённый reorder **Smoke после деплоя:** - `/site-finder?tab=market` → новый порядок: Competitors → BestLayouts → Velocity → MarketTrend → Pipeline24mo → SuccessRecommendation - Regression: `tab=overview` / `tab=land` без изменений Related: epic #271 item #6 (Wave 1).

lekss361 referenced this pull request

2026-05-17 14:10:40 +00:00

fix(sf-13): MarketTab above-the-fold headline + 3 KPI cards #294