Roadmap #307
Roadmap #307
Conversation
WalkthroughВ данном запросе на изменение добавлены новые элементы интерфейса и функциональность, связанные с отображением дорожной карты в клиенте Lost Paradise. В классе Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant EscapeMenu
participant RoadmapUI
participant RoadmapControl
participant RoadmapSystem
User->>EscapeMenu: Нажимает RoadmapButton
EscapeMenu->>RoadmapUI: ToggleRoadmap()
RoadmapUI->>RoadmapControl: RefreshUI()
RoadmapControl->>User: Отображает дорожную карту
RoadmapSystem->>RoadmapControl: Обновления фаз
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
github-actions
bot
added
Статус: Требуется одобрение
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 22
🧹 Outside diff range comments (1)
Content.Client/UserInterface/Systems/EscapeMenu/EscapeUIController.cs (1)
Line range hint
108-116: Предложение по улучшению управления URLРекомендуется перенести URL-адреса Discord и Boosty в конфигурацию, аналогично реализации WikiButton. Это упростит управление ссылками и их обновление в будущем.
Предлагаемые изменения:
- Добавьте новые параметры в CCVars:
public static string InfoLinksDiscord = "https://wiki.lost-paradise.space/discord"; public static string InfoLinksBoosty = "https://boosty.to/lost-paradise";
- Измените обработчики кнопок:
-_uri.OpenUri("https://wiki.lost-paradise.space/discord"); +_uri.OpenUri(_cfg.GetCVar(CCVars.InfoLinksDiscord)); -_uri.OpenUri("https://boosty.to/lost-paradise"); +_uri.OpenUri(_cfg.GetCVar(CCVars.InfoLinksBoosty));
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (11)
- Content.Client/Info/LinkBanner.cs (2 hunks)
- Content.Client/Options/UI/EscapeMenu.xaml (1 hunks)
- Content.Client/UserInterface/Systems/EscapeMenu/EscapeUIController.cs (2 hunks)
- Content.Client/_LostParadise/Roadmap/RoadmapControl.cs (1 hunks)
- Content.Client/_LostParadise/Roadmap/RoadmapUI.cs (1 hunks)
- Content.Server/_LostParadise/Roadmap/RoadmapSystem.cs (1 hunks)
- Content.Shared/_LostParadise/Roadmap/RoadmapComponent.cs (1 hunks)
- Content.Shared/_LostParadise/Roadmap/RoadmapPrototype.cs (1 hunks)
- Content.Shared/_LostParadise/Roadmap/RoadmapUpdateEvent.cs (1 hunks)
- Resources/Locale/ru-RU/_LostParadise/roadmap/roadmap.ftl (1 hunks)
- Resources/Prototypes/_LostParadise/Roadmap/roadmap.yml (1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
Resources/Locale/ru-RU/_LostParadise/roadmap/roadmap.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Prototypes/_LostParadise/Roadmap/roadmap.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.
🪛 yamllint
Resources/Prototypes/_LostParadise/Roadmap/roadmap.yml
[error] 1-1: trailing spaces
(trailing-spaces)
[error] 27-27: no new line character at the end of file
(new-line-at-end-of-file)
🔇 Additional comments (11)
Resources/Locale/ru-RU/_LostParadise/roadmap/roadmap.ftl (3)
1-2: Корректный перевод элементов меню!Переводы логически связаны и грамматически верны.
3-5: Корректный перевод индикаторов статуса!Все статусы переведены в правильном падеже и соответствуют контексту использования.
6-7: Корректный перевод заголовков!Переводы лаконичны и точно отражают назначение элементов интерфейса.
Content.Shared/_LostParadise/Roadmap/RoadmapComponent.cs (1)
7-9: Компонент корректно зарегистрирован и объявлен.Класс правильно помечен атрибутом
[RegisterComponent]и наследуется от базового классаComponent. Модификаторsealedпредотвращает наследование, что является хорошей практикой для компонентов.Content.Shared/_LostParadise/Roadmap/RoadmapPrototype.cs (1)
1-8: LGTM: Корректное объявление прототипаПравильное использование атрибута Prototype и наследование от IPrototype. Namespace соответствует структуре проекта.
Content.Server/_LostParadise/Roadmap/RoadmapSystem.cs (2)
1-9: Импорты и пространство имён корректны!Все необходимые зависимости присутствуют и правильно организованы.
14-19: Инициализация реализована корректно!Метод правильно вызывает базовую инициализацию и подписывается на необходимое событие.
Content.Client/Options/UI/EscapeMenu.xaml (1)
9-9: Корректная реализация кнопки дорожной картыКнопка правильно интегрирована в меню и следует установленным паттернам: публичный доступ, локализация текста и согласованное размещение в иерархии меню.
Content.Client/Info/LinkBanner.cs (1)
52-58: 🧹 Nitpick (assertive)Проверьте необходимость использования StyleBase.ButtonCaution
Стиль кнопки установлен как
ButtonCaution, что обычно используется для действий, требующих особого внимания пользователя. Убедитесь, что это соответствует UX-требованиям.Также рекомендуется:
- Добавить комментарий, объясняющий причину использования предупреждающего стиля
- Рассмотреть возможность вынесения обработчика нажатия в отдельный метод для улучшения читаемости
var roadmapButton = new Button { Text = Loc.GetString("server-info-roadmap-button"), - StyleClasses = { StyleBase.ButtonCaution }, }; -roadmapButton.OnPressed += _ => UserInterfaceManager.GetUIController<RoadmapUI>().ToggleRoadmap(); +roadmapButton.OnPressed += OnRoadmapButtonPressed; buttons.AddChild(roadmapButton); + +private void OnRoadmapButtonPressed(BaseButton.ButtonEventArgs args) +{ + UserInterfaceManager.GetUIController<RoadmapUI>().ToggleRoadmap(); +}Давайте проверим использование этого стиля в других местах:
Content.Client/_LostParadise/Roadmap/RoadmapControl.cs (1)
1-25: Корректная инициализация класса и конструктора!Структура класса хорошо организована, зависимости правильно внедрены через конструктор, а инициализация ProgressBar выполнена корректно.
Content.Client/UserInterface/Systems/EscapeMenu/EscapeUIController.cs (1)
67-71: Корректная реализация обработчика RoadmapButtonРеализация соответствует общему паттерну обработки кнопок в меню. Правильно организовано закрытие текущего окна перед отображением нового интерфейса.
|
не готова |
Evgencheg
added
Статус: Требуется изменения
|
я не могу понять, там в тестах ошибка какая-то, которая к моему роадмапу вообще ну никак не относится... |
Описание обновления:
Сделал роадмап.
Медиа
Проверки
Изменения
🆑 Farrellka