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

Adds new clothing #1193

Closed
wants to merge 8 commits into from
Closed

Conversation

HelmsDeepState
Copy link

@HelmsDeepState HelmsDeepState commented Apr 21, 2024

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

Добавляет четыре платья и две маски в слот противогаза
Платья в автодробе (вендомат в дормах и раздевалке), газики пока только админспаун

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

Больше красоты

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

Спрайты сделаны DiamondFox [discord]
d1
d6
d5
d4
d3
d2
d7

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

Протестировал локально

Changelog

🆑
add: Добавлены новые платья в Autodrobe
/:cl:

@github-actions github-actions bot added Merge Conflict Ну блять... 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! and removed Merge Conflict Ну блять... labels Apr 21, 2024
@ss220bot ss220bot added the 📜 CL валиден Этот чейнджлог будет успешно опубликован label Apr 21, 2024
@github-actions github-actions bot added the Merge Conflict Ну блять... label Apr 21, 2024
@AyIong
Copy link
Collaborator

AyIong commented Apr 22, 2024

Реши конфликты

Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts 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/clothing/code/clothing_vendors.dm Outdated Show resolved Hide resolved
modular_ss220/clothing/code/clothing_vendors.dm Outdated Show resolved Hide resolved
Comment on lines 206 to 250
/obj/item/clothing/suit/beatrice_dress
name = "розовое платье"
icon = 'modular_ss220/clothing/icons/object/suits.dmi'
icon_state = "beatricedress"
icon_override = 'modular_ss220/clothing/icons/mob/suits.dmi'

/obj/item/clothing/suit/black_idol_dress
name = "чёрное платье"
icon = 'modular_ss220/clothing/icons/object/suits.dmi'
icon_state = "black_idol_dress"
icon_override = 'modular_ss220/clothing/icons/mob/suits.dmi'


/obj/item/clothing/suit/blue_bright_dress
name = "светло-синее платье"
icon = 'modular_ss220/clothing/icons/object/suits.dmi'
icon_state = "blue_bright_dress"
icon_override = 'modular_ss220/clothing/icons/mob/suits.dmi'

/obj/item/clothing/suit/black_rose_dress
name = "чёрное вечернее платье"
icon = 'modular_ss220/clothing/icons/object/suits.dmi'
icon_state = "black_rose_dress"
icon_override = 'modular_ss220/clothing/icons/mob/suits.dmi'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Почему это suit, а не under, как другие платья в игре? Вряд ли есть причина, тогда стоит глянуть примеры в code\modules\clothing\under\dress.dm

@m-dzianishchyts
Copy link
Collaborator

m-dzianishchyts commented Apr 23, 2024

Важно еще знать, как оно все из этого сидит на других расах, особенно воксах. Как правило, нужно что-то двигать, обрезать, в том же файле есть пример

@github-actions github-actions bot removed the Merge Conflict Ну блять... label Apr 30, 2024
@HelmsDeepState
Copy link
Author

HelmsDeepState commented Apr 30, 2024

Уходим в драфт пока не будут спрайты на пришельцев. Остальные замечания приняты и поменяны

@HelmsDeepState HelmsDeepState marked this pull request as draft April 30, 2024 13:18
Copy link

This pull request seems to be stale as there have been no changes in 14 days, please make changes within 7 days or the PR will be closed. If you believe this is a mistake, please inform a development team member on Discord.

@github-actions github-actions bot added Stale ПР долго был не активен, и требует обновления. Merge Conflict Ну блять... and removed Stale ПР долго был не активен, и требует обновления. labels May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merge Conflict Ну блять... 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден Этот чейнджлог будет успешно опубликован
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants