Поднял алерты - #1180
Conversation
Walkthrough
ChangesРазмещение уведомлений
Estimated code review effort: 1 (Trivial) | ~3 минуты Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
🧹 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
📒 Files selected for processing (2)
Content.Client/UserInterface/Screens/SeparatedChatGameScreen.xamlContent.Client/UserInterface/Screens/SeparatedChatGameScreen.xaml.cs
|
Раньше было лучше 😔 |
может внешне и было лучше, но если их становится больше 7 то они начинают перекрывать куклу и уходить за экран |
Описание PR
поднял алерты чтобы их было хотя бы видно
Медиа
Список
Изменения
🆑 dettern