Skip to content

Conversation

@AndroBetel
Copy link
Member

@AndroBetel AndroBetel commented May 5, 2021

Описание изменений

https://github.com/TauCetiStation/TauCetiClassic/pull/6012/files переоткрытие
добавляет 4 вида мин
image
обычная - просто достаточно сильно бахает, есть в аплинке
шок - аля наэлектризованная дверь, пробивает человека разрядом электричества, заказывается в карго и есть у вардена
зажигательная - nuff said, есть в аплинке
эми - делает неплохой эми-импульс, производится в РнД.

после установки мины становятся полупрозрачными, что делает их незаметнее

разминируются мультитулом или выстрелом

Почему и что этот ПР улучшит

больше контента это круто

Авторство

я, спрайт сеуви

Чеинжлог

🆑

  • rscadd[link]: добавляет на станцию мины.

@TauKitty
Copy link
Contributor

TauKitty commented May 5, 2021

Changelog status: ✔️

@TauKitty TauKitty added Feature Новая фича Sprites labels May 5, 2021
@KIBORG04
Copy link
Member

KIBORG04 commented May 5, 2021

ясно переоткрытие

@KIBORG04 KIBORG04 added the Awaiting Review [Sprite Dep] Ожидается ревью от спрайт департамента label May 5, 2021
@KIBORG04 KIBORG04 requested a review from a team May 5, 2021 19:38
Co-authored-by: KIBORG04 <bossmira4@gmail.com>
AndroBetel and others added 2 commits May 5, 2021 22:52
Co-authored-by: KIBORG04 <bossmira4@gmail.com>
Co-authored-by: KIBORG04 <bossmira4@gmail.com>
@KIBORG04 KIBORG04 removed the Awaiting Review [Sprite Dep] Ожидается ревью от спрайт департамента label May 5, 2021
@Sakuya-Izayoi
Copy link
Contributor

Объясните, как это должно работать? Куда их плентить? Под тайлы шлюза? Или просто на пол, а сверху закидывать хламом чтобы не было видно? Или мина должна быть заметной чтобы человек понимал что туда лучше не соваться? Но ведь тогда её можно будет спокойно обойти или даже деактивировать.

@AndroBetel
Copy link
Member Author

if(anchored) icon_state = "[icon_state]armed" alpha = 45 else icon_state = initial(icon_state) alpha = 255

@KIBORG04
Copy link
Member

KIBORG04 commented May 5, 2021

он альфу уменьшает ей жоско. А так да, кладешь под аптечку или еще че


/obj/item/mine/emp/trigger_act(obj)
empulse(src, 2, 3)
qdel(src)
Copy link
Contributor

Choose a reason for hiding this comment

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

Если все мины удаляют себя возможно здесь имеет смысл вызывать ..(), а у родителя сделать кудел?

Copy link
Member

Choose a reason for hiding this comment

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

а можно обертку сделать для этой функции

Copy link
Member Author

Choose a reason for hiding this comment

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

а в чем конкретно разница будет между куделем и вызовом родителя?

Copy link
Contributor

Choose a reason for hiding this comment

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

Это более осмысленно. Удаляться после активации это свойство вообще всех мин, а не каких-то конкретных.

Copy link
Member Author

Choose a reason for hiding this comment

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

я либо кудель пишу, либо ссылку на кудель, так что разницы не особо наблюдаю

Copy link
Contributor

Choose a reason for hiding this comment

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

uses--
if(!uses) qdel

Copy link
Contributor

Choose a reason for hiding this comment

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

я не говорю что такое делать нужно, просто что дизайн такое ограничит. я ещё посмотрел историю сообщений, Бетель этот вопрос тоже поднимал

Copy link
Member Author

@AndroBetel AndroBetel May 5, 2021

Choose a reason for hiding this comment

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

так если тебе не нужно, чтобы мина удалялась, ты просто не пишешь ..() или qdel

Copy link
Member

Choose a reason for hiding this comment

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

А с оберткой можно было бы многоразовые делать

Copy link
Member

Choose a reason for hiding this comment

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

try_trigger() уже является оберткой для trigger_act(). Тебе достаточно убрать все кудели из trigger_act() и перенести их в самый низ try_trigger()

@AndroBetel
Copy link
Member Author

все сделал по ревью

@Rolt146
Copy link
Contributor

Rolt146 commented May 7, 2021

А есть ли реакция мин на мехи? Ибо я помню что некоторые вещи в механике не реагируют на мехов (например пена огнетушителя). Учтено ли это в пр?

@Sakuya-Izayoi
Copy link
Contributor

Напиши в ПРе сколько каждая мина стоит (и в какой ветке РнД последняя мина) и сколько они дамажат в цифрах

Copy link
Member

@KIBORG04 KIBORG04 left a comment

Choose a reason for hiding this comment

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

Вроде норм. Если что, я могу позже переделать это в компоненту и еще чего-то написать/переписать

@KIBORG04 KIBORG04 merged commit efe1df7 into TauCetiStation:master May 18, 2021
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request May 18, 2021
@AndroBetel AndroBetel deleted the welcometotheminefields branch June 3, 2021 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature Новая фича Sprites

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants