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

Voidsuit & hardsuit Infinity port #1717

Merged
merged 11 commits into from
Jan 22, 2024

Conversation

UEDCommander
Copy link

@UEDCommander UEDCommander commented Dec 27, 2023

ПР делает что-то умное, а именно - добавляет расовые спрайты, относящиеся к унати, скреллам и таяра, для войдсьютов и ИКС, что присутствовали на инфинити, но отсутствуют на Сьеррабее. Также добавляет отсутствующую на Сьеррабее возможность конвертировать скафандры СБ в риот-скафандры.

image image image
image image image
image image

close #1316

Чейнджлог

🆑UEDHighCommand
imageadd: Портированы расовые спрайты унати, скреллов для большинства войдсьютов
imageadd: Добавлены расовые спрайты унати, скреллов и таяра для ИКС ЛЭКа
rscadd: Унати, скреллы и таяра теперь могут пользоваться ИКС ЛЭКа 
rscadd: Риот-скафандр СБ возвращен в меню Security suit cycler
/🆑
  • Pull Request полностью завершен, мне не нужна помощь чтобы его закончить.
  • Я внимательно прочитал все свои изменения и багов в них не нашел.
  • Я запускал сервер со своими изменениями локально и все протестировал.

@github-actions github-actions bot added the ⛰️ Спрайты PR затрагивает спрайты label Dec 27, 2023
mods/hardsuits/README.md Outdated Show resolved Hide resolved
@UEDCommander UEDCommander marked this pull request as ready for review December 28, 2023 13:47
@UEDCommander UEDCommander requested a review from a team as a code owner December 28, 2023 13:47
Copy link

@SuhEugene SuhEugene left a comment

Choose a reason for hiding this comment

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

Хорошо. Сейчас у меня есть только две мелкие правки, но происходящее с ^= и |= меня пугает. Мне явно нужно найти свободное время чтобы хорошенько подумать и понять действительно ли это единственный способ решить задачу.

Решение интересное, но что-то мне подсказывает, что оно не лучшее и даже может навредить.

code/datums/item_modifiers/item_modifier.dm Outdated Show resolved Hide resolved
mods/hardsuits/_hardsuits.dme Outdated Show resolved Hide resolved
@UEDCommander
Copy link
Author

^= и |= решение, конечно, не единственное, но вероятно самое динамичное, которое можно сделать. В случае, если оффы полезут обновлять sprite sheet'ы, только при такой компоновке унатские листы не пострадают.
Есть ещё конечно lazyadd'ы, но судя по тому, что я так и не смог заставить таярский скин на ИКС ЛЭКа, который ими регулируется, работать, они не совсем удачно функционируют.

@SierraHelper SierraHelper added the 📜 CL валиден Ченджлог не содержит ошибок label Dec 30, 2023
Copy link

@SuhEugene SuhEugene left a comment

Choose a reason for hiding this comment

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

@LordNest
Copy link

Нужно отметить, что оно закрывает
#1316

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⛰️ Спрайты PR затрагивает спрайты 📜 CL валиден Ченджлог не содержит ошибок
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Старый спрайт: Спрайт инженерного скафа
4 participants