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

Красивые циферки урона #993

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

ThereDrD0
Copy link
Contributor

@ThereDrD0 ThereDrD0 commented Jan 1, 2025

Кратное описание

Старый визуал отвратителен, поэтому я сделал новый.

Теперь сами цифры урона намного меньше и позиция рандомится в небольшом радиусе от места прилета, пример в медиа.
Так же добавил возможность игнорировать некоторые типы урона, что настраивается в компоненте.

Игроки так же могут переключить пресет цифер, что будет появлятсья, а что нет

TODO

  • Циферки урона для структур типо стен
  • Возможность игроку выбрать в настройках направление, куда будет лететь попап. Да, не только вверх, но и в стороны
  • Выбор стороны в зависимости от положения игрока
  • Звук попадания?

По какой причине

Медиа(Видео/Скриншоты)

image

Content.client.2025.01.01.-.21.47.30.11.mp4

Проверки

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

Changelog

🆑 ThereDrD

  • tweak: Переработан визуал всплывающий цифр урона. Теперь они красивее
  • tweak: Добавлены всплывающие цифры урона для структур

@ThereDrD0 ThereDrD0 marked this pull request as draft January 1, 2025 04:34
@github-actions github-actions bot added size/M and removed size/S labels Jan 1, 2025
@ThereDrD0 ThereDrD0 marked this pull request as ready for review January 2, 2025 05:03
@@ -82,12 +83,22 @@ public ExtraTab()
var layoutLoc = Loc.GetString($"lobby-animation-{lobbyAnimation.ID}");
lobbyAnimations.Add(new OptionDropDownCVar<string>.ValueOption(lobbyAnimation.ID, layoutLoc));
}

Copy link
Collaborator

Choose a reason for hiding this comment

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

Зачем сколько кода
просто сделай так
Control.AddOptionCheckBox(SunriseCCVars.DamageOverlayEnableStructures, DamageOverlayEnabledStructuresCheckBox); Control.AddOptionCheckBox(SunriseCCVars.DamageOverlayEnableSelf, DamageOverlayEnabledSelfCheckBox);

Copy link
Collaborator

Choose a reason for hiding this comment

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

Зачем тут целый прототип ради 1 системы...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

А как ты предлагаешь без прототипа передать то, что включено у игроков? Создавать структуру под настройки оверлея?

Copy link
Collaborator

Choose a reason for hiding this comment

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

А как ты предлагаешь без прототипа передать то, что включено у игроков? Создавать структуру под настройки оверлея?

2 цвара... Структуры и урон себе, всё.

Copy link
Contributor Author

@ThereDrD0 ThereDrD0 Jan 4, 2025

Choose a reason for hiding this comment

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

А как ты предлагаешь без прототипа передать то, что включено у игроков? Создавать структуру под настройки оверлея?

2 цвара... Структуры и урон себе, всё.

Как на сервер-то передать это и как этим потом манипулировать. Через прототип все просто и красиво получается. Если менять прототип на чексбосы, нужно создавать структуру под настройки игрока или еще че. Посмотри как я _playerSettings использую и поймешь про что я

Copy link
Collaborator

Choose a reason for hiding this comment

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

Не нужно.

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