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

Atmospherics Overalls #2639

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Atmospherics Overalls #2639

wants to merge 4 commits into from

Conversation

oldschoolotaku
Copy link

@oldschoolotaku oldschoolotaku commented Oct 2, 2024 •
edited
Loading

Описание PR

Добавлена спецовка(overalls) атмосферного техника. Спрайты человека и иконки взяты с тгшки. Для скуфокса, змеюки и держания в руках нарисованы специально.

Почему / Баланс

Больше одежды атмосам. Из верхней одежды именно атмосов не учитывая риг и пожарку - куртка, которая является тупым респрайтом инженерной

Медиа

4yvak
skuf
zmeyz

Требования

Список изменений
🆑 oldschool_otaku

  • add: спецовка атмосферного техника

@github-actions github-actions bot added Status: Needs Review This PR requires new reviews before it can be merged Changes: No C# Code has not been changed Changes: Sprites Should be reviewed or fixed by people who are knowledgeable with spriting or visual design Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation labels Oct 2, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 2, 2024

RSI Diff Bot; head commit 15f7665 merging into 5d546cf
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Corvax/Clothing/OuterClothing/Coats/atmos_overalls.rsi

State Old New Status
equipped-OUTERCLOTHING-reptilian Added
equipped-OUTERCLOTHING-vox Added
equipped-OUTERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

@NotSoDana
Copy link
Contributor

offam

@oldschoolotaku
Copy link
Author

oldschoolotaku commented Oct 2, 2024 •
edited
Loading

у оффов стилистика спрайта должна быть абсолютно иная - смысл спрайтить сначала оффам, а уже потом сюда.

@p0etus
Copy link

p0etus commented Oct 3, 2024

offam

Корвакс уже не ванила, можно.

@Vonsant
Copy link
Collaborator

Vonsant commented Oct 3, 2024

image

@oldschoolotaku
Copy link
Author

82b5cv-2356303682

lzk228
lzk228 previously requested changes Oct 30, 2024
View reviewed changes
Copy link
Collaborator

@lzk228 lzk228 left a comment

Choose a reason for hiding this comment

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

не добавляем новые спрайты одежды до #2447

@PuroSlavKing
Copy link
Contributor

PuroSlavKing commented Nov 17, 2024 •
edited
Loading

не добавляем новые спрайты одежды до #2447

Как это связано с картой смещений? Уникальные спрайты под расу всё равно могут быть и есть, даже с учётом карты смещений, потому-что карта не может повторить сложные узоры, или правильную форму одежды (например клоуна), она просто заполняет, то чего не хватает.

@lzk228 lzk228 dismissed their stale review November 27, 2024 15:01

тщщи

@lzk228 lzk228 requested review from SonicHDC and removed request for ficcialfaint November 27, 2024 15:01
@SonicHDC
Copy link
Collaborator

Норм

@SonicHDC
Copy link
Collaborator

Но я фиксить баги не собираюсь. Решайте сами.

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

@SonicHDC SonicHDC SonicHDC approved these changes

@lzk228 lzk228 lzk228 left review comments

@Morb0 Morb0 Awaiting requested review from Morb0 Morb0 is a code owner

Assignees
No one assigned
Labels
Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation Changes: No C# Code has not been changed Changes: Sprites Should be reviewed or fixed by people who are knowledgeable with spriting or visual design S: Needs Review Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@oldschoolotaku @NotSoDana @p0etus @Vonsant @PuroSlavKing @SonicHDC @lzk228