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

Toggleable visibility of armor modules #810

Closed
wants to merge 28 commits into from

Conversation

CheBokJam
Copy link
Contributor

@CheBokJam CheBokJam commented Dec 12, 2024

Основные изменения

Теперь с помощью обычной краски для брони можно отключать видимость различных модулей для брони. НЕ РАБОТАЕТ С МОДУЛЯМИ ХРАНЕНИЯ (ВОЗМОЖНО ДОБАВЛЮ)
ПР будет в драфте, пока не пофиксятся все багулины (пжлста, пумохиде пожожда)
Как это выглядит:

dreamseeker_Bwd3OiqUQJ.mp4
  • Добавить такой же функционал для модулей хранения.
  • Исправить невозможность включения/выключения отображения модуля брони, если броня взята из лодаута.
  • Исправить баг с шлемами, когда после отключения и включения отображения модуля у них пропадают визоры (стёклышки у фреера, мимира,...) или изменяется целиком спрайт модуля (спрайты тура/сурта/... за авторством Саро меняются на оффовские варианты).
  • Исправить, что некоторые модули (балдур и ещё какой-то) не имею этого функционала.
  • Исправить неработающую корректно смену спрайтов у модулей шлема с активацией (сварочный модуль, бинокли,...)
  • Добавить возможность красить модули брони.
  • Исправить отображение камы, чтобы пушка была поверх неё, а не торчала будто бы из ж.
  • Спрятанный модуль при сохранении лодаута и последующем взятии лодаута остаётся скрытым.
  • Добавить возможность красить модули брони при нажатии краской по броне.
  • Прописать флаги для корректной работы покраски и скрытия модулей в отдельный для этого файл.
  • Рассортировать спрайты и говно в файле модулей: спрайты модулей хранения и модулей брони в разные дми; убрать кучу ненужных наследований icon_state, item_state и icon; рассортировать модули по телу и голове; убрать нахер ненужные спрайты "active" коробок и, соответственно, код, использующий эти спрайты.
  • Перевести документацию по использованию грейскейла/гиперскейла и по возможности дополнить, описав некоторые моменты.
  • Вернуть сенсорному модулю шлема спрайт при активации
  • Насрано в код, убрать за собой.

Как это улучшит игру

Дрип важнее жизни епта.

Ченджлог

:cl:
add: Добавил новые механики или изменил геймплей.
add: Добавил больше вещей.
del: Удалил старые вещи.
qol: Сделал что-то более простым в использовании.
balance: Изменил баланс некоторых вещей.
fix: Исправил некоторые вещи.
sound: Добавил/Удалил/Изменил пару звуков.
image: Добавил/Удалил/Изменил некоторые спрайты.
map: Изменил составляющую карты.
spellcheck: Исправил несколько опечаток.
code: Изменил некую составляющую кода.
refactor: Переделал код в лучшую сторону.
config: Изменил настройки конфига.
admin: Что-то изменил в механиках для админов.
server: Что-то, в чем должны разбираться владельцы сервера.
/:cl:

@Tatarla
Copy link
Collaborator

Tatarla commented Dec 13, 2024

Давай тогда и покраску модулей

@CheBokJam
Copy link
Contributor Author

Давай тогда и покраску модулей

Тяжело житя... Я попробую, но ничего не обещаю.

@CheBokJam

This comment was marked as resolved.

@CheBokJam

This comment was marked as resolved.

Copy link
Contributor

github-actions bot commented Jan 4, 2025

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 3 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale Автор ушёл на завод. label Jan 4, 2025
@Helg2 Helg2 added Stale Автор ушёл на завод. and removed Stale Автор ушёл на завод. labels Jan 4, 2025
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 3 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale Автор ушёл на завод. label Jan 15, 2025
@Helg2 Helg2 removed the Stale Автор ушёл на завод. label Jan 15, 2025
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 3 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale Автор ушёл на завод. label Jan 23, 2025
@github-actions github-actions bot closed this Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale Автор ушёл на завод.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants