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

Add Temporary waddle status effect #1527

Merged

Conversation

Kirus59
Copy link
Collaborator

@Kirus59 Kirus59 commented Aug 6, 2024

Описание PR

Добавил статус эффект ковыляния(как от клоунских туфель) для фрезона (трекер).

При надевании клоунских туфель - интенсивность прыжков увеличивается до значений в WaddleWhenWornComponent, при их снятии - уменьшается до значений в TemporaryWaddleComponent

Медиа

2024-08-06.14-11-19.1.1.mp4

Проверки

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

Изменения

🆑 kirus59

  • add: Добавлено ковыляние (как при ношении клоунских туфель) при вдыхании фрезона.

@Kirus59 Kirus59 marked this pull request as draft August 6, 2024 08:32
@Kirus59 Kirus59 marked this pull request as ready for review August 6, 2024 09:19
@Kirus59
Copy link
Collaborator Author

Kirus59 commented Aug 6, 2024

Пофиксил также анимацию ковыляния во время крита/смерти куклы.

@Kirus59
Copy link
Collaborator Author

Kirus59 commented Aug 6, 2024

Че мне дионы и слаймы тесты руинят.....
image
image

@Kirus59
Copy link
Collaborator Author

Kirus59 commented Aug 6, 2024

Уменьшил интенсивность прыжков во время покачивания в 2 раза по просьбе заказчика.

@DexlerXD
Copy link

DexlerXD commented Aug 6, 2024

@Kirus59 через EntityEffects не пробовал сделать? Её оффы реворкнули, теперь лава через неё работает, например (систему лавы полностью из игры выпилили).

@Kirus59
Copy link
Collaborator Author

Kirus59 commented Aug 6, 2024

@Kirus59 через EntityEffects не пробовал сделать? Её оффы реворкнули, теперь лава через неё работает, например (систему лавы полностью из игры выпилили).

Я не понял как через EntityEffects удалить компонент анимации, когда эффект от реагента заканчивается.
Поэтому решил использовать статус эффект.

Copy link

Choose a reason for hiding this comment

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

перенести в отдельный пр

@DexlerXD DexlerXD merged commit 60d899b into SerbiaStrong-220:master Aug 6, 2024
14 checks passed
@Kirus59 Kirus59 deleted the TemporaryWaddleStatusEffect branch August 6, 2024 20:04
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