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

Silent shoes #2146

Merged
merged 9 commits into from
Oct 31, 2024
Merged

Silent shoes #2146

merged 9 commits into from
Oct 31, 2024

Conversation

lexaSvarshik
Copy link

@lexaSvarshik lexaSvarshik commented Oct 23, 2024

Описание PR

По задаче
Добавлены (пусть и костыльно) бесшумные ботинки. При ходьбе по любым поверхностям (лужи, ковры, решетки, обычный пол) не издают абсолютно никаких звуков.
sneaky

Todo

  • Уникальный спрайт для них (будет потом)
    или
  • Сделать хамелеонами
  • Подкорректировать цену при надобности (1/2 ТК)

Медиа

изображение

Проверки

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

Изменения

🆑 Svarshik

  • add: Добавлены бесшумные ботинки в аплинк!
  • tweak: Ходьба ниндзи теперь бесшумна.

@github-actions github-actions bot added the Changes: Localization Изменение затронуло файлы ".ftl" label Oct 23, 2024
@Kirus59
Copy link
Collaborator

Kirus59 commented Oct 24, 2024

Зачем этот костыль?
Почему бы просто не добавлять компонент FootstepModifier?

А если от наследуется от Base, то почему бы не сделать свой Base но без этого компонента?

@Kirus59 Kirus59 self-assigned this Oct 24, 2024
@Kirus59 Kirus59 changed the title Silent shoes [WiP] Silent shoes Oct 24, 2024
@lexaSvarshik
Copy link
Author

lexaSvarshik commented Oct 24, 2024

Зачем этот костыль? Почему бы просто не добавлять компонент FootstepModifier?

Если его не добавлять, то у нас проигрываются дефолтные звуки ботинок.

А если от наследуется от Base, то почему бы не сделать свой Base но без этого компонента?

Нигде в Parent / Base не используется компонент FootstepModifier или какой-либо другой компонент, задающий звук шагов.

Звук шагов приходит отсюда Content.Shared\Maps\ContentTileDefinition.cs
изображение

Ну и система Content.Shared\Movement\Systems\SharedMoverController.cs
изображение

@Kirus59
Copy link
Collaborator

Kirus59 commented Oct 24, 2024

А да, оно использует дефолтные звуки ходьбы если нет модификатора на ботинках, увы

@UrPrice UrPrice marked this pull request as draft October 27, 2024 17:13
@github-actions github-actions bot added the Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" label Oct 28, 2024
@lexaSvarshik
Copy link
Author

А чет быстро драфт поставили, вчера обсудили, я доделал.
Вкратце - сейчас боты хамелеон. Когда будет для них уникальный крутой спрайт - поменяем.

@lexaSvarshik lexaSvarshik marked this pull request as ready for review October 28, 2024 13:38
@lexaSvarshik lexaSvarshik changed the title [WiP] Silent shoes Silent shoes Oct 28, 2024
@UrPrice
Copy link
Collaborator

UrPrice commented Oct 28, 2024

А чет быстро драфт поставили, вчера обсудили, я доделал. Вкратце - сейчас боты хамелеон. Когда будет для них уникальный крутой спрайт - поменяем.

image
Действительно

@UrPrice UrPrice requested a review from Kirus59 October 30, 2024 22:31
@Kirus59
Copy link
Collaborator

Kirus59 commented Oct 31, 2024

@lexaSvarshik попробуй то, о чем я выше писал

@lexaSvarshik
Copy link
Author

lexaSvarshik commented Oct 31, 2024

@lexaSvarshik попробуй то, о чем я выше писал

Ну кстати, нормально работает. Сейчас тогда поменяю

@Kirus59 Kirus59 merged commit b2de865 into SerbiaStrong-220:master Oct 31, 2024
13 checks passed
@lexaSvarshik lexaSvarshik deleted the silent-boots branch October 31, 2024 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants