Skip to content
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

Add smes bound ui #736

Merged
merged 2 commits into from
Feb 15, 2024
Merged

Add smes bound ui #736

merged 2 commits into from
Feb 15, 2024

Conversation

Skriler
Copy link

@Skriler Skriler commented Feb 14, 2024

Описание PR

Добавил bound ui для СМЭС

Медиа

Content Client_F4xHcs3TBn

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑

  • add: Добавлен интерфейс на ЛКМ для СМЭС.

@github-actions github-actions bot added the Changes: UI Изменение затронуло файлы ".xaml" label Feb 14, 2024
@DexlerXD
Copy link

@Ady4ik мнение по интерфейсу?

@Ady4ik
Copy link

Ady4ik commented Feb 15, 2024

Выглядит круто, но надо бы добавить мб вывод накопленного значение в мВт/Вт?

@github-actions github-actions bot added the Changes: Localization Изменение затронуло файлы ".ftl" label Feb 15, 2024
@Skriler
Copy link
Author

Skriler commented Feb 15, 2024

@Ady4ik обновил скрин, так подходит?

@Ady4ik
Copy link

Ady4ik commented Feb 15, 2024

Идеально
@DexlerXD можно мерджить

@DexlerXD DexlerXD merged commit be15081 into SerbiaStrong-220:master Feb 15, 2024
12 checks passed
@Skriler Skriler deleted the smes-ui-fix branch February 17, 2024 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение затронуло файлы ".ftl" Changes: UI Изменение затронуло файлы ".xaml"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants