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

[Resprite] Chaplain #2563

Merged
merged 21 commits into from
Sep 7, 2024
Merged

Conversation

MureixloI
Copy link
Contributor

@MureixloI MureixloI commented Sep 4, 2024

Описание PR

Изменение спрайтов некоторой одежды роли священника.
Так как большинству участникам комьюинити понравилась идея, чтобы изменить спрайты.
Медиа

364537057-13cca4c5-f89d-44b8-9c48-3f7bcfd57b43.gif
Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑 MureixloL

  • tweak: Изменены спрайты!

@MureixloI MureixloI requested a review from Morb0 as a code owner September 4, 2024 20:26
@MureixloI
Copy link
Contributor Author

@SonicHDC

@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 labels Sep 4, 2024
Copy link
Contributor

github-actions bot commented Sep 4, 2024

RSI Diff Bot; head commit 1ad72d4 merging into 625d70d
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Clothing/Head/Hoods/chaplain.rsi

State Old New Status
equipped-HELMET Modified
icon Modified

Resources/Textures/Clothing/Head/Hoods/cult.rsi

State Old New Status
equipped-HELMET Modified
icon Modified

Resources/Textures/Clothing/Head/Hoods/nun.rsi

State Old New Status
equipped-HELMET Modified
icon Modified

Resources/Textures/Clothing/OuterClothing/Misc/cultrobes.rsi

State Old New Status
equipped-OUTERCLOTHING-reptilian Modified
equipped-OUTERCLOTHING Modified
icon Modified

Resources/Textures/Clothing/OuterClothing/Misc/nunrobe.rsi

State Old New Status
equipped-OUTERCLOTHING Modified
icon Modified

Resources/Textures/Clothing/Uniforms/Jumpskirt/chaplain.rsi

State Old New Status
equipped-INNERCLOTHING Modified
icon Modified
inhand-left Modified
inhand-right Modified

Resources/Textures/Clothing/Uniforms/Jumpsuit/chaplain.rsi

State Old New Status
equipped-INNERCLOTHING-reptilian Modified
equipped-INNERCLOTHING Modified
icon Modified
inhand-left Modified
inhand-right Modified

Resources/Textures/Clothing/Uniforms/Jumpsuit/monastic_robe_dark.rsi

State Old New Status
equipped-INNERCLOTHING-reptilian Modified
equipped-INNERCLOTHING Modified
icon Modified

Resources/Textures/Clothing/Uniforms/Jumpsuit/monastic_robe_light.rsi

State Old New Status
equipped-INNERCLOTHING-reptilian Modified
equipped-INNERCLOTHING Modified
icon Modified

Edit: diff updated after 1ad72d4

@Sergan31
Copy link

Sergan31 commented Sep 4, 2024

Я серьезно, у нас есть тот же скрап, чьи спрайты основаны на спрайтах оффов, но вместо респрайта их у нас очередной респрайт того, что и без респрайта не особо выделяется

@Meguneri
Copy link
Contributor

Meguneri commented Sep 4, 2024

Жду респрайт скрапа день 18

@BolloTea
Copy link

BolloTea commented Sep 4, 2024

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

@lzk228
Copy link
Collaborator

lzk228 commented Sep 4, 2024

image
нет нет и ещё раз нет
это не подгон под стиль, это хуйня, которая меняется смысл одежды

@MureixloI
Copy link
Contributor Author

image нет нет и ещё раз нет это не подгон под стиль, это хуйня, которая меняется смысл одежды

Если честно, я тут подумала, то реально лучше просто ретекстур сделать нынешний

@MureixloI MureixloI marked this pull request as draft September 4, 2024 23:32
@MureixloI MureixloI marked this pull request as ready for review September 5, 2024 08:16
@BolloTea
Copy link

BolloTea commented Sep 5, 2024

image нет нет и ещё раз нет это не подгон под стиль, это хуйня, которая меняется смысл одежды

Тут да, да и это больше нуарной одежде бы подошло. Я в целом обобщил чтобы в пустую не бухтели на спрайтеров, что хоть что-то делают. Если и бухтеть, то по делу и указывать на недостатки или предлагать альтернативы

@BolloTea
Copy link

BolloTea commented Sep 7, 2024

Чтобы рубашка белая не пропала, можно будет заменить нуарный наряд или сделать альтернативу

@SonicHDC SonicHDC merged commit a65e756 into space-syndicate:master Sep 7, 2024
11 of 12 checks passed
@SonicHDC
Copy link
Collaborator

SonicHDC commented Sep 7, 2024

Увы пропала

@BolloTea
Copy link

BolloTea commented Sep 9, 2024

Ждём рубашку в следующем пиаре

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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 Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants