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

Fix shoes reptilian #2398

Closed
wants to merge 1 commit into from
Closed

Fix shoes reptilian #2398

wants to merge 1 commit into from

Conversation

Litogin
Copy link
Contributor

@Litogin Litogin commented Jul 10, 2024

Описание PR

Теперь у вульп и унатхов спрайты туфель отображаются правильно
фикс этого бага: #2314
Медиа

Вульпа:

2024-07-10.09-56-15.mp4

Унатх:

2024-07-10.09-56-35.mp4

Проверки

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

Изменения

🆑 Litogin

  • fix: Исправлено отображение спрайтов у вульп и унатхов

@Litogin Litogin requested a review from Morb0 as a code owner July 10, 2024 06:58
@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 Jul 10, 2024
@github-actions GitHub Actions
Copy link
Contributor

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

Resources/Textures/Clothing/Shoes/color.rsi

State Old New Status
contrastedsoles-equipped-FEET-reptilian Added
equipped-FEET-reptilian Added
soles-equipped-FEET-reptilian Added

lzk228
lzk228 requested changes Jul 10, 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.

используй дисплейсмент карты

@Litogin
Copy link
Contributor Author

Litogin commented Jul 10, 2024

используй дисплейсмент карты

А разница? Все равно изменение в прототипе будут такие же

@lzk228
Copy link
Collaborator

lzk228 commented Jul 10, 2024

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

@lzk228 lzk228 added Status: Awaiting Changes This PR needs its reviews addressed or changes to be made in order to be merged and removed Status: Needs Review This PR requires new reviews before it can be merged labels Jul 10, 2024
@TheShuEd
Copy link
Contributor

учитесь дисплейсмент картам, да. Век перерисовки тысяч спрайтов под расу подошел к концу, и вам стоит принять это. Лазик, не мержи это!

@TheShuEd
Copy link
Contributor

используй дисплейсмент карты

А разница? Все равно изменение в прототипе будут такие же

нихрена, ты делаешь 1 дисплейсмент карту, и все спрайты (и будущие тоже) автоматически адаптируются

@Litogin
Copy link
Contributor Author

Litogin commented Jul 10, 2024

используй дисплейсмент карты

А разница? Все равно изменение в прототипе будут такие же

нихрена, ты делаешь 1 дисплейсмент карту, и все спрайты (и будущие тоже) автоматически адаптируются

А там рофл что оффы сделали хуйню, и я не думаю что оно сработает

@lzk228
Copy link
Collaborator

lzk228 commented Jul 10, 2024

ещё одно слово вякнешь в сторону разработчиков

@TheShuEd
Copy link
Contributor

ещё одно слово вякнешь в сторону разработчиков

И Лазик тебя [ДАННЫЕ УДАЛЕНЫ]

@Litogin
Copy link
Contributor Author

Litogin commented Jul 11, 2024 •
edited
Loading

Закрываю Пр, т.к буду делать Дисплесмент карты для унатхов и вульп (Одно и тоже строение тела)
P.r смысла пока починка данный баг, пока не будет реализовано отключение карты у определённых предметов

@Litogin Litogin closed this Jul 11, 2024
@Litogin Litogin deleted the fix-shoes branch July 11, 2024 19:26
@Litogin Litogin restored the fix-shoes branch July 12, 2024 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lzk228 lzk228 lzk228 requested changes

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

Assignees
No one assigned
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: Awaiting Changes This PR needs its reviews addressed or changes to be made in order to be merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Litogin @lzk228 @TheShuEd