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

Feat: Delta SM Space Drop #685

Merged
merged 20 commits into from
Nov 14, 2023

Conversation

GREG-MAP
Copy link

@GREG-MAP GREG-MAP commented Nov 12, 2023

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

  1. Добавляет механику сброса СМ в космос, а именно под солярами на край карты при взрыве которого он даже их не задевает. Так же добавляет кнопку сброса и инструкцию по ее использованию.

  2. Переделка зоны содержания СМ и прибытия под систему сброса.

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

Позволяет в критической ситуации, а именно когда целостность СМ доходит до 10 или менее процентов, сбросить его в космос да бы не разносить весь атмос и не ебать с ним голову остаток раунда инженерам. Тем более большинство Капитанов после взрыва СМ сразу вызывают шатл хотя при взрыве СМ спокойно можно продолжать раунд.

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

image

2023-11-14 18 23 31

default.mp4
1.mp4

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

Сотню раз запускал на локалке и тестил все начиная от пути запуска до взрыва

Changelog

🆑
add: Систему сброса СМ
imageadd: Иконка кнопки сброса СМ
/:cl:

@github-actions github-actions bot added Merge Conflict Ну блять... 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один. 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! Керберос Этот ПР изменил что-то на Керберосе (DeltaStation) labels Nov 12, 2023
Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Идея: Казуальщина, лично мне было бы все равно, но на Кибериаду прошу такое не нести (по крайней мере пока что). Требуется голосование среди игроков Дельты.

Кодстайл: ОК

Кнопка сброса СМ: Абсолютно не уникальный спрайт, я бы подумал что это выключатель света. Спрайт стоило бы перерисовать.
Опасная, стоит переделать её под кнопку с доступом, и добавить может какую-нибудь закрывашку, как например в первой Халфе.

image

modular_ss220/sm_space_drop/code/sm_button.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_button.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/sm_space_drop.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/sm_space_drop.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_mass_driver.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_button.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_button.dm Outdated Show resolved Hide resolved
@GREG-MAP GREG-MAP force-pushed the Delta-SM-SpaceDropSystem branch from f09e4d1 to 4aa58b0 Compare November 12, 2023 21:20
@github-actions github-actions bot removed the Merge Conflict Ну блять... label Nov 12, 2023
Copy link

@Legendaxe Legendaxe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Комната ожидания переделки лишней копипасты кода в новую унаследованную кнопку

modular_ss220/sm_space_drop/code/sm_button.dm Outdated Show resolved Hide resolved
@AyIong
Copy link
Collaborator

AyIong commented Nov 12, 2023

Идея: Казуальщина, лично мне было бы все равно, но на Кибериаду прошу такое не нести (по крайней мере пока что). Требуется голосование среди игроков Дельты.

Так ставь дизлайк если идея не нравится, я вон въебал

@GREG-MAP GREG-MAP marked this pull request as draft November 12, 2023 21:54
@GREG-MAP GREG-MAP marked this pull request as ready for review November 13, 2023 21:28
@AyIong AyIong added the 🌽 Отклонено Увы, вы были посланы нахуй. НЕ МЕРЖИТЬ, пожалуйста. label Nov 13, 2023
@AyIong AyIong removed the 🌽 Отклонено Увы, вы были посланы нахуй. НЕ МЕРЖИТЬ, пожалуйста. label Nov 14, 2023
@AyIong
Copy link
Collaborator

AyIong commented Nov 14, 2023

Днём внимательно гляну

@AyIong AyIong marked this pull request as draft November 14, 2023 00:38
@GREG-MAP GREG-MAP marked this pull request as ready for review November 14, 2023 00:39
Copy link
Collaborator

@PhantornRU PhantornRU left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Глянул. В принципе работает, но можно сделать лучше и поинтереснее. Универсальнее, на случай если данная реализация используется не только в одной закрытой комнате

@github-actions github-actions bot added the 🔉 Звук Услада для ушей. label Nov 14, 2023
modular_ss220/sm_space_drop/code/sm_button_NEW.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_button_NEW.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_button_NEW.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_button_NEW.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/paper_sm_info.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/paper_sm_info.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/paper_sm_info.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/paper_sm_info.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/paper_sm_info.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/paper_sm_info.dm Outdated Show resolved Hide resolved
Copy link
Collaborator

@PhantornRU PhantornRU left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ThumbsUp, но запрос на улучшение, центрирование и отступы все еще остался.

@dj-34 dj-34 changed the title Delta-SM-Space-Drop Feat: Delta SM Space Drop Nov 14, 2023
@AyIong
Copy link
Collaborator

AyIong commented Nov 14, 2023

image

modular_ss220/sm_space_drop/code/sm_button_NEW.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_button_NEW.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/paper_sm_info.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_button_NEW.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_button_NEW.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_button_NEW.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/paper_sm_info.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_button_NEW.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_button_NEW.dm Outdated Show resolved Hide resolved
modular_ss220/sm_space_drop/code/sm_button_NEW.dm Outdated Show resolved Hide resolved
Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вроде норм, можно отправлять на голосование игрокам.
По хорошему еще проверить правильно ли работает логгирование админское

Copy link
Collaborator

@AyIong AyIong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FUCKIN FINALLY

@Legendaxe Legendaxe merged commit ec837b6 into ss220club:master Nov 14, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден 🔉 Звук Услада для ушей. 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один. Керберос Этот ПР изменил что-то на Керберосе (DeltaStation)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants