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

[TGUI] Jukebox #214

Merged
merged 6 commits into from
Oct 5, 2023
Merged

[TGUI] Jukebox #214

merged 6 commits into from
Oct 5, 2023

Conversation

AyIong
Copy link
Collaborator

@AyIong AyIong commented Sep 20, 2023

Что этот PR делает

Музыкальный автомат и диско-шар переведены на TGUI.
Сейчас это по-большей части порт с ТГ, однако планирую следующее:

  • Добавление прочности и корректная смена иконки на поломанный вариант.
  • Потребление электричества при воспроизведении музыки.
  • Прерывание музыки если автомат сломали или сел АПЦ.
  • Работа музыки при потере из виду автомата а так же направленный звук. (Либо я Skill Issue, либо это никак не сделать нормально. Слышно на всю карту, звук не затухает до 0. В общем похуй
  • Починить отключение музыки и танцев при выключении в настройках.
  • Перенос всего в модуль.

Без понятия смогу ли я это реализовать со своими навыками в кодерстве, но надеюсь да.
В противном случае оставлю как есть и вещь так и останется только на ЦК.

Почему это хорошо для игры

МУЗЫКА, ТУСОВКИ, ДИС-КО-ТЕ-КА!!!

Изображения изменений

dreamseeker_23-29-32_.08.09.2023.mp4

Тестирование

А как иначе?

Changelog

🆑
add: Музыкальный автомат и куча музыки к нему!
del: Админы больше не могут загрузить музыку в диско-шар/музыкальный автомат. Увы.
tweak: Добавлять музыку для музыкального автомата - проще. Но только для хоста Jokerge
/:cl:

@github-actions github-actions bot added 💾 Изменение конфига Ф-ф-фуриор...? TGUI Кто-то решил потрогать интерфейсы. labels Sep 20, 2023
@github-actions github-actions bot added Merge Conflict Ну блять... and removed Merge Conflict Ну блять... labels Sep 21, 2023
@AyIong AyIong marked this pull request as draft October 5, 2023 11:36
@github-actions github-actions bot added 🔉 Звук Услада для ушей. 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! and removed Merge Conflict Ну блять... labels Oct 5, 2023
@github-actions github-actions bot added the 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один. label Oct 5, 2023
@AyIong AyIong marked this pull request as ready for review October 5, 2023 11:44
@Furrior Furrior merged commit cc56e0f into ss220club:master Oct 5, 2023
18 checks passed
Furrior pushed a commit that referenced this pull request Oct 22, 2023
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то
может пойти не так. -->
<!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. -->

Музыкальный автомат и диско-шар переведены на TGUI.
Сейчас это по-большей части порт с ТГ, однако планирую следующее:

- [x] Добавление прочности и корректная смена иконки на поломанный
вариант.
- [x] Потребление электричества при воспроизведении музыки.
- [x] Прерывание музыки если автомат сломали или сел АПЦ.
- [ ] Работа музыки при потере из виду автомата а так же направленный
звук. (Либо я Skill Issue, либо это никак не сделать нормально. Слышно
на всю карту, звук не затухает до 0. В общем похуй
- [x] Починить отключение музыки и танцев при выключении в настройках.
- [x] Перенос всего в модуль.

Без понятия смогу ли я это реализовать со своими навыками в кодерстве,
но надеюсь да.
В противном случае оставлю как есть и вещь так и останется только на ЦК.

<!-- Вкратце опишите изменения, которые вносите. -->
<!-- Опишите **все** изменения, так как противное может сказаться на
рассмотрении этого PR'а! -->
<!-- Если вы исправляете Issue, добавьте "Fixes #1234" (где 1234 - номер
Issue) где-нибудь в описании PR'а. Это автоматически закроет Issue после
принятия PR'а. -->

МУЗЫКА, ТУСОВКИ, ДИС-КО-ТЕ-КА!!!
<!-- Опишите, почему, по вашему, следует добавить эти изменения в игру.
-->

https://github.com/ss220club/Paradise-SS220/assets/69762909/60856e42-ae4c-4ae9-aefc-5c4a116cded6

<!-- Если вы не меняли карту или спрайты, можете опустить эту секцию.
Если хотите, можете вставить видео. -->

А как иначе?
<!-- Как вы тестировали свой PR, если делали это вовсе? -->

:cl:
add: Музыкальный автомат и куча музыки к нему!
del: Админы больше не могут загрузить музыку в диско-шар/музыкальный
автомат. Увы.
tweak: Добавлять музыку для музыкального автомата - проще. Но только для
хоста Jokerge
/:cl:

<!-- Оба :cl:'а должны быть на месте, что-бы чейнджлог работал! Вы
можете написать свой ник справа от первого :cl:, если хотите. Иначе
будет использован ваш ник на ГитХабе. -->
<!-- Вы можете использовать несколько записей с одинаковым префиксом
(Они используются только для иконки в игре) и удалить ненужные. Помните,
что чейнджлог должен быть понятен обычным игроком. -->
<!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы
можете исключить всю секцию. -->

---------

Co-authored-by: DGamerL <[email protected]>
BR54FF pushed a commit to BR54FF/Paradise-SS220 that referenced this pull request Oct 23, 2023
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то
может пойти не так. -->
<!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. -->

Музыкальный автомат и диско-шар переведены на TGUI.
Сейчас это по-большей части порт с ТГ, однако планирую следующее:

- [x] Добавление прочности и корректная смена иконки на поломанный
вариант.
- [x] Потребление электричества при воспроизведении музыки.
- [x] Прерывание музыки если автомат сломали или сел АПЦ.
- [ ] Работа музыки при потере из виду автомата а так же направленный
звук. (Либо я Skill Issue, либо это никак не сделать нормально. Слышно
на всю карту, звук не затухает до 0. В общем похуй
- [x] Починить отключение музыки и танцев при выключении в настройках.
- [x] Перенос всего в модуль.

Без понятия смогу ли я это реализовать со своими навыками в кодерстве,
но надеюсь да.
В противном случае оставлю как есть и вещь так и останется только на ЦК.

<!-- Вкратце опишите изменения, которые вносите. -->
<!-- Опишите **все** изменения, так как противное может сказаться на
рассмотрении этого PR'а! -->
<!-- Если вы исправляете Issue, добавьте "Fixes ss220club#1234" (где 1234 - номер
Issue) где-нибудь в описании PR'а. Это автоматически закроет Issue после
принятия PR'а. -->

МУЗЫКА, ТУСОВКИ, ДИС-КО-ТЕ-КА!!!
<!-- Опишите, почему, по вашему, следует добавить эти изменения в игру.
-->

https://github.com/ss220club/Paradise-SS220/assets/69762909/60856e42-ae4c-4ae9-aefc-5c4a116cded6

<!-- Если вы не меняли карту или спрайты, можете опустить эту секцию.
Если хотите, можете вставить видео. -->

А как иначе?
<!-- Как вы тестировали свой PR, если делали это вовсе? -->

:cl:
add: Музыкальный автомат и куча музыки к нему!
del: Админы больше не могут загрузить музыку в диско-шар/музыкальный
автомат. Увы.
tweak: Добавлять музыку для музыкального автомата - проще. Но только для
хоста Jokerge
/:cl:

<!-- Оба :cl:'а должны быть на месте, что-бы чейнджлог работал! Вы
можете написать свой ник справа от первого :cl:, если хотите. Иначе
будет использован ваш ник на ГитХабе. -->
<!-- Вы можете использовать несколько записей с одинаковым префиксом
(Они используются только для иконки в игре) и удалить ненужные. Помните,
что чейнджлог должен быть понятен обычным игроком. -->
<!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы
можете исключить всю секцию. -->

---------

Co-authored-by: DGamerL <[email protected]>
@AyIong AyIong deleted the jukebox-rebased branch November 6, 2023 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💾 Изменение конфига Ф-ф-фуриор...? 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден 🔉 Звук Услада для ушей. TGUI Кто-то решил потрогать интерфейсы. 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants