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

unhardcode night and thermal vision clothing, fix squadron helmet sprite #269

Closed
wants to merge 3 commits into from

Conversation

pheenty
Copy link
Contributor

@pheenty pheenty commented Nov 13, 2024

О PR

  1. Анхардкод ThermalVisionClothing и NightvisionCloting, теперь они работают везде, кроме карманов, а не только на глазах. Пока что нигде не используется, в будущем нужно для Feature: термальное зрение, встроенное в шлем ЭС #267
  2. Как забавное следствие, фиксит удаление ThermalVision/Nightvision при замене одного предмета с ThermalVisionClothing/NightvisionClothing на другой.
    (как оно работает сейчас - заменил термальный монокль на термальный визор - термальное зрение пропало, надо на секунду снимать визор и надевать назад чтобы его вернуть).
    Потому что почему-то предмет сначала надевается в слот, а только потом из слота убирается предмет, который в этом слоте был.
  3. Прочие мелкие улучшения для повышения качества кода.
  4. Фикс отсутствия текстурки у шлема ЭС.

Почему / Баланс

фиксы

Технические детали

  1. ThermalVision и Nightvision теперь хранят sources. Для встроенных способностей это null, а если оно даётся одеждой, то лист с uid этой одежды. Если после снятия длина листа нулевая, компонент удаляется.

Медиа

image

  • Я добавил к этому PR скриншоты/видео, демонстрирующие его изменения в игре, или этот PR не требует демонстрации в игре

Changelog

Copy link
Contributor

github-actions bot commented Nov 13, 2024

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

Resources/Textures/Clothing/Head/Hardsuits/deathsquad.rsi

State Old New Status
icon-flash Removed

Edit: diff updated after f325c32

@pheenty
Copy link
Contributor Author

pheenty commented Nov 13, 2024

Чёт тесты посыпались. Странно, щас буду смотреть

UPD. в ходе переноса с #268 потерялось 1 переименование файла и 1 буква s в прототипе. Бывает.

@doublechest0
Copy link
Contributor

Думаю надо использовать EquipmentHudSystem

@doublechest0
Copy link
Contributor

Плохая реализация.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants