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

BOUNTY: Возможность носить перчатки при потере 1 кисти #6007

Closed
wants to merge 5 commits into from

Conversation

AmShegars
Copy link

Описание

Позволяет носить перчатки при потери 1 кисти, спрайт меняется в зависимости от того какая кисть утрачена.

Ссылка на предложение/Причина создания ПР

Баунти https://discord.com/channels/617003227182792704/1292557454646837349

Демонстрация изменений

https://cdn.discordapp.com/attachments/1292557454646837349/1293896929167540338/2024-10-10_16-23-33.mp4?ex=67090aeb&is=6707b96b&hm=562b051bd0d89da784699fe7b4a5112289412f6b159a461c14e5bea9ef0510d0&

Тесты

-Надеть перчатки
-Рубануть себя по кисти катаной

bam
@github-actions github-actions bot added the DMI Affects DMI files label Oct 10, 2024
code/modules/clothing/clothing.dm Outdated Show resolved Hide resolved
@@ -313,9 +314,13 @@ BLIND // can't see anything
SPECIES_DRASK = 'icons/mob/clothing/species/drask/gloves.dmi'
)

/obj/item/clothing/gloves/Initialize(mapload)
. = ..()
basic_item_state = item_state
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
Author

Choose a reason for hiding this comment

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

без неё item_state превращается в item_state_r_r_r_r_r_r_r_r_r_r. Решил вот так.

Copy link
Contributor

Choose a reason for hiding this comment

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

без неё item_state превращается в item_state_r_r_r_r_r_r_r_r_r_r. Решил вот так.

Можно же просто initial(item_state) использовать. Нет?

@github-actions github-actions bot added the Merge Conflict This doesn't seem right label Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DMI Affects DMI files Merge Conflict This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants