Repository navigation
fix: исправить мобильный редактор, приглашения и уведомления связи - #405
Merged
Merged
Conversation
This was referenced Oct 10, 2026
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.
Что изменено
Мобильный редактор проекта и приглашения больше не имеют пересечений кнопок, двойной рамки поиска и узкого поля даты. Карточки приглашений центрируются, строки внимания в аналитике не выглядят как полосы прогресса, пустая кнопка контактов программы скрыта.
Вместо двух повторяющихся ошибок соединения показано одно закрываемое сообщение о недоступности чата. Подключение восстанавливается автоматически; после восстановления сообщение исчезает.
Зачем
Закрыть десять дефектов, обнаруженных при smoke-test dev на iPhone, с сохранением восстановленного desktop ≥1000 px.
Реализация
<1000px: проекты, редактор/цели/партнёры, AvatarControl, Input/Textarea, RegionSelect, приглашения и аналитика.ChatRealtimeRepositoryбольше не создаёт второе уведомление.ConnectionStatusToastServiceвладеет одним snackbar на период сбоя;WebsocketServiceпредоставляет статус восстановления и отменяет retry при явном закрытии сессии.API
HTTP/WebSocket contracts, payload, авторизация и очередь исходящих сообщений не менялись. Изменены транспортный lifecycle и UX его состояния; бизнес-правила проектов, программ и приглашений сохранены.
UI
Использованы текущие Mont, палитра и компоненты. Эталон — пользовательские screenshots и текущий dev
706a574eпосле восстановления desktop в #403. Новый desktop-дизайн не вводится. Новая close-кнопка применяется только к закрываемому уведомлению.Permissions
Server-side permissions и scope объектов не менялись: backend/API diff отсутствует. Существующие проверки ролей и отправки приглашений сохранены.
Проверки
Локальная среда: Windows, Node.js 24, Chrome с fixtures. В репозитории нет workflow для pull_request: GitHub проверки dev запускаются только по push в
devпосле merge.npm run test:vitest:ci: 2016 tests / 407 files, exit 0. Три существующих legacy exclusions перечислены в отчёте.npm run build:social:prodи финальная development-сборка: exit 0.npm run lint:ts: 0 errors, 6 существующих warnings вне diff.npm run format:check: exit 0 на staged Git snapshot с LF и тем же CI-конфигом.Acceptance criteria
Не проверено
Реальные iOS/Android, Safari/WebKit, загрузка файлов, live API и сетевой обрыв на dev. Локальные браузерные проверки используют синтетические API/WS и неизменённые файлы build через Playwright route, чтобы исключить сбои локального HTTP/proxy. После merge повторить smoke-test на физических устройствах, включая background/resume и восстановление чата.
Риски / migration
Backend/data migration не требуется. Transport lifecycle покрыт тестами очереди, retry, явного закрытия и восстановления; реальные device/network сценарии остаются этапом acceptance на dev. Merge и deploy в этот PR workflow не входят.