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.

Sign up for GitHub

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

Jump to bottom

Breath scarf #702

Merged
merged 12 commits into from
Nov 22, 2023
Merged

Breath scarf #702

merged 12 commits into from
Nov 22, 2023

Conversation

FlutterSnedDraw
Copy link

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

Добавляет шарфик-дыхательную маску в донат,который одевается в слот маски и является более модным способом спасти себя при разгерметизации. Выполняется по просьбе murken4ik.

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

Теперь даже при разгерметизации можно быть модным.

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

image

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

Проверил наличие на локалке.

Changelog

🆑
add: Шарфик для дыхания.
/:cl:

@github-actions github-actions bot added the 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! label Nov 14, 2023
dj-34
dj-34 requested changes Nov 14, 2023
View reviewed changes
modular_ss220/clothing/code/mask.dm Outdated Show resolved Hide resolved
modular_ss220/clothing/code/mask.dm Outdated Show resolved Hide resolved
@dj-34
Copy link
Collaborator

dj-34 commented Nov 14, 2023

сук, еще икондифф сломался, не проверить

@FlutterSnedDraw
Copy link
Author

сук, еще икондифф сломался, не проверить

Внес изменения,еще что-то?

dj-34
dj-34 approved these changes Nov 14, 2023
View reviewed changes
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.

Нет возможности проверить иконки (икондифф сдох), по коду и кодстайлу все ок должно быть

dj-34
dj-34 requested changes Nov 14, 2023
View reviewed changes
Copy link
Collaborator

@dj-34 dj-34 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Хотя я кое-что заметил. Нет иконки шарфа в руках и нет иконки шарфа в инвентаре/на полу.

@FlutterSnedDraw
Copy link
Author

Хотя я кое-что заметил. Нет иконки шарфа в руках и нет иконки шарфа в инвентаре/на полу.

image
Хм,на полу и в инвентаре у меня есть спрайт ,а вот в руках нет спрайта ,потому что у нас никакие шарфы и маски не имеют спрайт в руках,да и выглядеть в руках она скорее как черное месиво.

@dj-34
Copy link
Collaborator

dj-34 commented Nov 15, 2023

Хотя я кое-что заметил. Нет иконки шарфа в руках и нет иконки шарфа в инвентаре/на полу.

image Хм,на полу и в инвентаре у меня есть спрайт ,а вот в руках нет спрайта ,потому что у нас никакие шарфы и маски не имеют спрайт в руках,да и выглядеть в руках она скорее как черное месиво.

это не спрайт "на полу", это отвратительный юз иконки с моба

PhantornRU
PhantornRU requested changes Nov 16, 2023
View reviewed changes
modular_ss220/clothing/code/mask.dm Outdated Show resolved Hide resolved
@FlutterSnedDraw
Copy link
Author

Хотя я кое-что заметил. Нет иконки шарфа в руках и нет иконки шарфа в инвентаре/на полу.

image Хм,на полу и в инвентаре у меня есть спрайт ,а вот в руках нет спрайта ,потому что у нас никакие шарфы и маски не имеют спрайт в руках,да и выглядеть в руках она скорее как черное месиво.

это не спрайт "на полу", это отвратительный юз иконки с моба

image
переделал

@github-actions github-actions bot added the Merge Conflict Ну блять... label Nov 17, 2023
@dj-34
Copy link
Collaborator

dj-34 commented Nov 19, 2023

Разконфликть

dj-34
dj-34 previously requested changes Nov 19, 2023
View reviewed changes
modular_ss220/loadout/code/donor.dm Show resolved Hide resolved
@AyIong
Copy link
Collaborator

AyIong commented Nov 21, 2023

Расконфликтуй и добавь отступ как диджей написал

@github-actions github-actions bot removed the Merge Conflict Ну блять... label Nov 21, 2023
@FlutterSnedDraw
Copy link
Author

Расконфликтуй и добавь отступ как диджей написал

done

@AyIong AyIong requested review from dj-34 and PhantornRU November 21, 2023 22:25
PhantornRU
PhantornRU approved these changes Nov 22, 2023
View reviewed changes
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.

13 строчек - 100 правок для шмоточки

@AyIong AyIong dismissed dj-34’s stale review November 22, 2023 08:06

Сделано

@AyIong AyIong merged commit 3043bfd into ss220club:master Nov 22, 2023
11 of 13 checks passed
@FlutterSnedDraw
Copy link
Author

И не говори :garold:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AyIong AyIong AyIong approved these changes

@PhantornRU PhantornRU PhantornRU approved these changes

@dj-34 dj-34 Awaiting requested review from dj-34

Assignees
No one assigned
Labels
🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@FlutterSnedDraw @dj-34 @AyIong @PhantornRU @ss220bot