-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bluespace harvester #957
base: master
Are you sure you want to change the base?
bluespace harvester #957
Conversation
RSI Diff Bot; head commit b3f4c71 merging into 74f4d40 Resources/Textures/ADT/Bluespace/Crates/biological_secure.rsi
Resources/Textures/ADT/Bluespace/Crates/destruction_secure.rsi
Resources/Textures/ADT/Bluespace/Crates/industrial_secure.rsi
Resources/Textures/ADT/Bluespace/Crates/limited_secure.rsi
Resources/Textures/ADT/Bluespace/Crates/technological_secure.rsi
Resources/Textures/ADT/Bluespace/harvester.rsi
Resources/Textures/ADT/Objects/Structures/Power/bluespace_transitor.rsi
|
Все сделал. На локалке проблем нет - по крайне мере видимых проблем не обнаружил. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
В шарпы сильно не вглядывался, пока только это
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Такс, я пробежался глазами по шарпам. Проблемы в серверной части не такие критические, но очень желательно поправить UI. Если что-то будет непонятно, ткни меня в дисе, помогу
@@ -0,0 +1,6 @@ | |||
<Control xmlns="https://spacestation14.io"> | |||
<GridContainer Margin = "2 0 0 0" Columns="2"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Зачем использовать GridContainer, когда есть BoxContainer с Orientation="Horizontal"? Лучше изменить, удобней будет в будущем редактировать
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Лучше сделать UI меньше по горизонтали. Он выглядит растянуто и пусто сейчас
- Где Margin, Лебовски? Добавь отступы от краёв окна, а то выглядит плохо
- Прогресс бар уходит в пустоту, надо как-либо ограничить его
- Может стоит добавить кнопки для повышения/понижения уровня?
[GenerateTypedNameReferences] | ||
public sealed partial class BluespaceHarvesterMenu : FancyWindow | ||
{ | ||
private readonly BluespaceHarvesterBoundUserInterface _owner; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Вместо макарон использования классов друг другом, используй экшны.
Например:
В меню - public Action<int>? SetLevelAction;
для создания и SetLevelAction?.Invoke(level);
для его использования.
В функции Open в BUI добавляешь _window.SetLevelAction += args => Blabla
, и теперь каждый раз, когда используется SetLevelAction, будет вызываться Blabla (функция, которая имеет аргумент соответствующего типа, что и экшн)
Описание PR
Добавление Блюспейс-добытчики с Bluspace Harvester [Fixed], и мелкое исправление Bluespace update c CorvaxNext. Проверял ещё месяц назад на другом сервере, проблем никаких не обнаружил.
Встречайте машину, которая потребляет энергию сотен станций, Bluespace Harvester!
Новый источник интересных и полезных материалов. Он использует энергию для создания коробок с предметами разной полезности. Но если его использовать неправильно (или если вмешаются агенты синдиката), он может стать угрозой для жизни всей станции!
Более подробно можно узнать по ссылке выше и по этим видео:
390329650-e07e38c2-27d3-4b04-b824-b412ad073be7.mp4
390329604-d4a20083-3926-41e8-b22d-44a60070f68f.mp4
🆑 CrimeMoot