Skip to content

Поднял алерты - #1180

Open
islambrosgood wants to merge 1 commit into
WWhiteDreamProject:masterfrom
islambrosgood:alert
Open

Поднял алерты#1180
islambrosgood wants to merge 1 commit into
WWhiteDreamProject:masterfrom
islambrosgood:alert

Conversation

@islambrosgood

Copy link
Copy Markdown
Contributor

Описание PR

поднял алерты чтобы их было хотя бы видно


Медиа

Список

изображение


Изменения

🆑 dettern

  • tweak: Алерты подняты

@coderabbitai

coderabbitai Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

AlertsUI размещён в новом вертикальном контейнере TopRightContainer. Контейнер привязан к правому верхнему углу с отступом 10.

Changes

Размещение уведомлений

Layer / File(s) Summary
Контейнер и позиционирование уведомлений
Content.Client/UserInterface/Screens/SeparatedChatGameScreen.xaml, Content.Client/UserInterface/Screens/SeparatedChatGameScreen.xaml.cs
AlertsUI перемещён в TopRightContainer. В конструкторе контейнер установлен в позицию TopRight с отступом 10.

Estimated code review effort: 1 (Trivial) | ~3 минуты

Suggested reviewers: deathb4defeat, spatison

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed Заголовок описывает основное изменение: перемещение алертов в видимую область интерфейса.
Description check ✅ Passed Описание прямо связано с изменениями и объясняет цель перемещения алертов.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
Content.Client/UserInterface/Screens/SeparatedChatGameScreen.xaml.cs (1)

27-27: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Удалите устаревшую привязку Alerts.

AlertsUI теперь является дочерним элементом TopRightContainer в Content.Client/UserInterface/Screens/SeparatedChatGameScreen.xaml, Line 38-40. Вызов SetAnchorAndMarginPreset(Alerts, LayoutPreset.CenterRight, margin: 10) на Line 25 больше не должен использоваться. В стандартной реализации RobustToolbox attached-свойства якорей обрабатывает LayoutContainer, а BoxContainer самостоятельно раскладывает дочерние элементы. (github.com)

Оставьте позиционирование только для TopRightContainer. Проверьте это поведение на версии engine проекта.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@Content.Client/UserInterface/Screens/SeparatedChatGameScreen.xaml.cs` at line
27, Удалите устаревший вызов SetAnchorAndMarginPreset для Alerts в инициализации
SeparatedChatGameScreen, поскольку AlertsUI теперь раскладывается внутри
TopRightContainer. Оставьте только позиционирование TopRightContainer через
SetAnchorAndMarginPreset и убедитесь, что поведение соответствует версии engine
проекта.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@Content.Client/UserInterface/Screens/SeparatedChatGameScreen.xaml.cs`:
- Line 27: Удалите устаревший вызов SetAnchorAndMarginPreset для Alerts в
инициализации SeparatedChatGameScreen, поскольку AlertsUI теперь раскладывается
внутри TopRightContainer. Оставьте только позиционирование TopRightContainer
через SetAnchorAndMarginPreset и убедитесь, что поведение соответствует версии
engine проекта.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 3dcbda74-b348-4173-b80e-e79c9978346f

📥 Commits

Reviewing files that changed from the base of the PR and between ff38ac4 and 782dc3e.

📒 Files selected for processing (2)
  • Content.Client/UserInterface/Screens/SeparatedChatGameScreen.xaml
  • Content.Client/UserInterface/Screens/SeparatedChatGameScreen.xaml.cs

@Lachklen

Lachklen commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Раньше было лучше 😔

@islambrosgood

Copy link
Copy Markdown
Contributor Author

Раньше было лучше 😔

может внешне и было лучше, но если их становится больше 7 то они начинают перекрывать куклу и уходить за экран

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants