DEV: улучшить карточки участников и поиск - #377
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.
Карточка каталога участников стала читаемее: имя и фамилия выделены, вторичная информация отделена, навыки выровнены слева и занимают максимум две строки с точным
+N. Аватар сохраняет 70×70 px и прежнее положение относительно рамки. Ширина карточки 156 px, одинаковая высота увеличена с 180 до 248 px; CTA ведёт в существующий профиль единственной ссылкой.На 1024 px воспроизведено исходное переполнение сетки. Локальная media-правка оставляет 3 колонки в диапазоне 1000–1279 px; на широком desktop — 4, на mobile — прежняя одна. Статистика, общие карточки проектов/команд и остальные фильтры не переработаны.
Поиск: устранена потеря последних символов быстрого ввода (debounce вместо throttle только для fullname), нормализованы пробелы, resolver учитывает запрос из URL, обновляется total пагинации, поздняя страница старого поиска не загрязняет новую выдачу.
Base DEV:
bdb5cf4780be0daa3e441d6b152511fcbe638d4eHead:
9a1036f8de34bc05c73e18e85a43b494a9a5df76Зависимость: backend #754 исправляет регистр кириллицы и полное имя в прежнем публичном
/auth/public-users/. Read-only API-проверка обнаружила проблему и на DEV, и на PROD. Angular не загружает всех пользователей для локальной фильтрации, API-контракт не меняется.Проверки (Node 20.20.2):
npm run test:ci -- --pool=forks: 1791/1791, 387 файлов, exit 0, без unhandled errors;npm run lint:ts: exit 0, 6 предупреждений в незатронутых файлах;npm run build:prod: exit 0, предупреждения CommonJS;git diff --check: GREEN;Отчёт, измерения и все скриншоты
Ограничения: скриншоты локальные, на синтетических данных. Авторизованной live-сессии нет; после исправления DEV/PROD не проверены, deploy не выполнялся. В локальном браузере исходное предупреждение NG0912 воспроизводится до/после. На Windows штатный build:sprite с одинарными кавычками создаёт пустой sprite: автоматически изменённый файл восстановлен из HEAD, в PR не включён; screenshots используют исходный валидный sprite. Ошибки не подавлялись, инфраструктура тестов/сборки не изменялась.
React, статистика, зависимости, workflows и backend-код в этом PR не затронуты. Только Draft → dev. Merge/deploy не выполнять.
Состояние зависимого backend CI: Lint и обычный CI/Tests прошли; отдельный PostgreSQL workflow упал до тестов на установке зависимостей Poetry/pip (TypeError при чтении версии setuptools). Это ограничение remote CI указано в #754; локально backend 887/887. Инфраструктура вне scope этой задачи, её изменения или ручной rerun не выполнялись.