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

imageadd: inhand gloves #6504

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

Conversation

BoshyG
Copy link
Contributor

@BoshyG BoshyG commented Jan 31, 2025

Описание

Инхенд спрайты для цветных перчаток, армированных, капитанских и сватовских

Причина создания ПР / Почему это хорошо для игры

По просьбе кисика сделал часть инхенд спрайтов для некоторых основных перчей а так же тех что я добавилял.

Демонстрация изменений

изображение
изображение
изображение

Тесты

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

@github-actions github-actions bot added the DMI Affects DMI files label Jan 31, 2025
@BoshyG
Copy link
Contributor Author

BoshyG commented Jan 31, 2025

https://discord.com/channels/617003227182792704/734823601110515882/1335009432509091953 - ссылка с показом что все ок работает в дискорде

@BoshyG
Copy link
Contributor Author

BoshyG commented Feb 1, 2025

Уэээ част ьспрайтов удалилась

@BoshyG BoshyG closed this Feb 1, 2025
@BoshyG BoshyG reopened this Feb 1, 2025
@BeebBeebBoob
Copy link
Member

Конфликт + Если будет возможность, то сделай отдельный файл для этого gloves_lefthand.dmi gloves_righthand.dmi

@github-actions github-actions bot added the Merge Conflict This doesn't seem right label Feb 2, 2025
@BoshyG
Copy link
Contributor Author

BoshyG commented Feb 2, 2025

Конфликт + Если будет возможность, то сделай отдельный файл для этого gloves_lefthand.dmi gloves_righthand.dmi

А куда скинуть файл сюда отдельно? Или коммитом в директорию?

@FlitchTime
Copy link
Contributor

А куда скинуть файл сюда отдельно? Или коммитом в директорию?

Коммитом конечно

@BoshyG BoshyG reopened this Feb 15, 2025
@github-actions github-actions bot removed the Merge Conflict This doesn't seem right label Feb 15, 2025
@BoshyG
Copy link
Contributor Author

BoshyG commented Feb 15, 2025

Сделал перчи отдельным файлом

@BeebBeebBoob
Copy link
Member

Надо было ещё приписать в коде для нового файла спрайтов иначе они не будут показываться

@BeebBeebBoob BeebBeebBoob added the Автор покинул Автор покинул чат и не желает больше работать label Mar 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DMI Affects DMI files Автор покинул Автор покинул чат и не желает больше работать
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants