Skip to content

DEV: fix profile navigation state and remaining UI regressions - #312

Merged
TThanos3000 merged 1 commit into
devfrom
fix/dev-profile-navigation-ui-regressions
Aug 30, 2026
Merged

DEV: fix profile navigation state and remaining UI regressions#312
TThanos3000 merged 1 commit into
devfrom
fix/dev-profile-navigation-ui-regressions

Conversation

@TThanos3000

@TThanos3000 TThanos3000 commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Что исправлено

  • Каталог вакансий использует backend applicant-state flags (hasResponded, canRespond, canManageResponses) для CTA. Руководитель получает переход manageResponses=true, кандидат — sendResponse=true, участник проекта и уже откликнувшийся пользователь видят только «подробнее».
  • Query parameter manageResponses открывает существующее окно откликов только после проверки canManageResponses; без права список не загружается.
  • Полное описание проекта действительно отображается после «Подробнее»: дублирующий CSS line-clamp удалён.
  • Общий SkillsGroupComponent переведён на контролируемую активную категорию: переключение A → B работает одним кликом и сохраняет выбранные навыки во всех текущих местах использования.
  • В правой панели навыков добавлен верхний отступ 14px; выбранная специализация обозначается фиолетовой точкой.
  • Модалки библиотеки навыков закрываются явным крестиком и backdrop через существующий openChange.
  • «Обо мне» всегда отображается полностью; ненужный ExpandService-flow удалён.
  • Длинные названия проектов больше не режутся pipe по числу символов: полный текст остаётся в DOM, визуально используется двухстрочный clamp.
  • isProfileEmpty теперь вычисляется как computed от текущего просматриваемого пользователя. Удалён stale-flow, который вычислял пустоту по предыдущему route-user. Загрузка новостей использует switchMap и очищает прежний route-state перед новым запросом.

Dependency

Backend catalog-state contract: PROCOLLAB-github/api#719 — merged into dev.

Проверки

  • Targeted UI/service tests: 43/43 passed.
  • Full npm run test:ci: 327 files, 1020/1020 tests passed.
  • npm run lint:ts: passed, 0 errors; 6 pre-existing warnings.
  • Scoped Prettier: passed.
  • Stylelint changed SCSS: passed.
  • npm run build:prod: passed; only existing Angular/Sass/budget warnings.
  • git diff --check: passed.

Scope

Изменения ограничены Angular application code и tests. React, backend, workflows, deploy, Docker и Tilda не изменялись.

@TThanos3000
TThanos3000 marked this pull request as ready for review August 30, 2026 20:26
@TThanos3000
TThanos3000 merged commit bee34fc into dev Aug 30, 2026
@TThanos3000
TThanos3000 deleted the fix/dev-profile-navigation-ui-regressions branch August 30, 2026 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant