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.

Sign up for GitHub

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

Jump to bottom

Jukebox update #2018

Merged
merged 6 commits into from
Oct 9, 2024
Merged

Jukebox update #2018

merged 6 commits into from
Oct 9, 2024

Conversation

Surani1
Copy link

@Surani1 Surani1 commented Oct 5, 2024 •
edited
Loading

Описание PR

Добавлена ползунок звука у Jukebox, теперь можно спокойно настраивать его громкость.

Медиа
хуй

Проверки

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

Изменения
🆑 Stalengd Surani

  • add: Добавлена полоска громкости жукбокса

@github-actions github-actions bot added Changes: UI Изменение затронуло файлы ".xaml" Changes: Localization Изменение затронуло файлы ".ftl" labels Oct 5, 2024
@UrPrice
Copy link
Collaborator

UrPrice commented Oct 5, 2024

@Ady4ik DNM пока не добавят локальную настройку громкости жукбокса, согласен?

@Surani1
Copy link
Author

Surani1 commented Oct 5, 2024

@Ady4ik DNM пока не добавят локальную настройку громкости жукбокса, согласен?

Я разрешаю поплакать, настройка громкости сделана

@UrPrice
Copy link
Collaborator

UrPrice commented Oct 5, 2024

@Ady4ik DNM пока не добавят локальную настройку громкости жукбокса, согласен?

Я разрешаю поплакать, настройка громкости сделана

*Сделана настройка громкости объекта, а не игрока-клиента, чтобы у себя убрать громкость музыки.

@Surani1 Surani1 marked this pull request as ready for review October 6, 2024 11:50
@Surani1
Copy link
Author

Surani1 commented Oct 6, 2024

@DexlerXD

DexlerXD
DexlerXD suggested changes Oct 6, 2024
View reviewed changes
Resources/Locale/en-US/jukebox/jukebox-menu.ftl Outdated Show resolved Hide resolved
Resources/Locale/ru-RU/jukebox/jukebox-menu.ftl Outdated Show resolved Hide resolved
Content.Client/Audio/Jukebox/JukeboxMenu.xaml Show resolved Hide resolved
@Surani1 Surani1 requested review from DexlerXD and stalengd October 7, 2024 14:14
@Surani1
Copy link
Author

Surani1 commented Oct 7, 2024

@DexlerXD

@Surani1 @stalengd
Update Content.Server/Audio/Jukebox/JukeboxSystem.cs
39c1249 
Co-authored-by: Stalen <33173619+stalengd@users.noreply.github.com>
@DexlerXD DexlerXD merged commit 4f9821a into SerbiaStrong-220:master Oct 9, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stalengd stalengd stalengd requested changes

@DexlerXD DexlerXD DexlerXD approved these changes

Assignees
No one assigned
Labels
Changes: Localization Изменение затронуло файлы ".ftl" Changes: UI Изменение затронуло файлы ".xaml" Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Surani1 @UrPrice @stalengd @DexlerXD