DEV: добавить модальное чтение новостей в ленте - #374
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.
В общей ленте короткие новости больше не занимают карточку высотой 420 px. Новости людей и проектов показывают компактное превью; клик, Enter, Space или «Подробнее» открывают полный текст, изображения и вложения в локальном окне поверх существующего app-modal.
База:
e2c6d00dded1e026a9ec2850612ea8822f75e9e3(DEV после #373).Head:
258a3ea370c409877cf4673c7f7c4de55f2c3163.Что изменено:
images().lengthвместо длины signal-функции.Только Angular. Backend, React, API-контракт, runtime API-слой, пагинация/category counts #372, фильтры, legacy detail/profile, shared modal, editor, dependencies, workflows и test harness не изменены. В API-каталоге добавлен только regression spec существующего UI state.
Проверки:
npm run test:ci: exit 1, известный NG0401 в setupTestBed до выполнения тестов.npm run test:ci -- --pool=forks: 1775/1775, 385 файлов, exit 0, без unhandled errors.npm run lint:ts, Stylelint изменённых SCSS, Prettier изменённых файлов,npm run build:prod,git diff --check: exit 0. Pre-commit выполнен штатно.Реальная локальная browser-проверка на 1440/1280/1024/390 px: нет horizontal overflow; desktop 2 колонки, mobile 1. На 1440 высоты рядов 266,5 / 314 / 358 px, соседи и CTA выровнены. Thumbnail 120 × 67,5 px. Modal максимум 880 × 810 px, mobile 366 × 759,59 px, header остаётся видимым при прокрутке 80 абзацев. Портрет, квадрат, 4:3, 16:9 и панорама показаны целиком; пять картинок переключаются.
Полный отчёт, геометрия и все 18 скриншотов · Команды и ограничения.
Портрет, landscape, длинный текст + landscape, mobile feed, mobile modal, mobile длинный текст + портрет.
Приёмка выполнена на настоящих Angular-компонентах с синтетическим fixture-facade, без live DEV и пользовательских данных. В консоли errors нет, NG0912 присутствует на base и head. Внешнее скачивание, переходы live DEV и содержимое системного clipboard отдельно не подтверждены; делегирование действий покрыто тестами.
Draft → dev. Merge/deploy выполняет пользователь.