PROD: обновить страницу участников - #380
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Селективный перенос страницы «Участники» из DEV #377/#378/#379. Поиск больше не теряет последний введённый текст: нормализует пробелы, сохраняет fullname в URL, обновляет пагинацию и отбрасывает устаревшие ответы. Карточки показывают один основной навык +N, mobile использует отдельное окно существующих фильтров.
База и источники
0cd4bb85ba8368695a3b0859e83630cb715067e5.5525d40a6b9810b963b543ef056ed308fc87ea72.3c67203663e2b4dd50b8e72df3d5fb53390c287f.63e67b3b59513f7a82f23122c4a51440251b280f.96a7a676f7d47194fa105fb1748a5bfc28b5b544.Отличие от DEV и геометрия
Единственное runtime-отличие от финальных source-файлов DEV — уменьшение общей высоты member card 248 → 210 px. Измеренный skill→CTA gap 53 → 15 px, без ручного поднятия кнопки. Avatar position unchanged: 70×70 px, прежние CSS top/left, измеренный Y относительно рамки −34 px. Desktop width 156 px, CTA 32 px, имя/secondary info и skill row 22 px сохранены.
Реальная Angular fixture с семью синтетическими случаями проверена на 1440/1280/1024/768/414/390/375. Все карточки 210 px; gap 15 px; CTA выровнены; horizontal overflow = 0. Сетка 4/4/3/2/1/1/1, mobile без правого sidebar, статистика ниже списка. Проверены открытие фильтров, Escape и возврат фокуса. Console errors fixture: 0.
Скриншоты до/после, геометрия и воспроизводимая fixture.
Проверки
Node 20.20.2,
npm ciуспешно.npm run test:ci -- --pool=forks projects/social_platform/src/app/api/member/facades/members-info.service.spec.ts projects/social_platform/src/app/ui/pages/members/: 20/20, 5 файлов, exit 0.npm run test:ci -- --pool=forks: 1902/1902, 395 файлов, exit 0, без NG0401/unhandled errors.npm run lint:ts: exit 0, 6 существующих предупреждений.npx stylelintизменённых SCSS: exit 0.npx prettier --checkизменённых TS/HTML/SCSS/MD/JSON: exit 0.npm run build:prod: exit 0, существующие Sass/CommonJS/budget warnings.git diff --check: OK. Штатный pre-commit (Stylelint + ESLint) пройден.Границы и ограничения
Перенесены 17 исходных файлов только api/member и ui/pages/members; остальные изменения — документация/синтетическая QA fixture/скриншоты. Statistics logic untouched, React untouched, shared app-modal unchanged. API models, остальные filters, dependencies, lockfile, workflows, DEV не менялись.
Fixture использует реальные компоненты и стили, но подменяет данные/фасад выдачи и не воспроизводит всю оболочку office. Live PROD/DEV поиск не проверен: безопасной авторизованной сессии нет; реальные данные не менялись. Поиск покрыт тестами фасада и PostgreSQL API в backend PR.
Windows-ограничение штатного build:sprite: одинарные кавычки glob дают пустой generated sprite; после успешного build только этот файл восстановлен из HEAD и не входит в diff. Preview проверен с исходным корректным sprite. Build scripts не изменены.
Merge/deploy не выполнялись.