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

Добавил фигурку зловещего кота #19

Closed
wants to merge 6 commits into from

Conversation

PCCartem
Copy link

@PCCartem PCCartem commented Dec 10, 2024

О PR

Добавил фигурку зловещего кота из мема "Бу, испугался?"

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

Шутки ради, для веселья.

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

Добавил спрайт, прописал фразы, внес изменения в ящик фигурок и библиотеку, для того, чтобы был шанс появления фигурки. Добавил к фигуркам Stories компонент, который позволяет им говорить. Если не прописать SpeakOnUse, при нажатии, как и раньше, ничего не происходит.

Медиа

2024-12-09_17-54-15
2024-12-09_17-54-04
2024-12-09_17-53-21
2024-12-09_17-53-06

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

Changelog

🆑 variarti

  • add: Добавлена фигурка зловещего кота из мема "Бу! Испугался?"
  • tweak: Теперь все фигурки Stories (потенциально) могут говорить.

Comment on lines 36 to 46
- type: entity
parent: BaseFigurine
id: ToyFigurineBooCat
name: sinister cat figure
description: A figurine depicting a sinister cat that usually lives in the darkness of technical tunnels.
components:
- type: Sprite
state: boo_cat
- type: SpeakOnUse
pack: FigurinesBooCat

Copy link
Contributor

Choose a reason for hiding this comment

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

Это надо в Stories папку

@@ -112,6 +112,7 @@
- ToyFigurineParamedic
- ToyFigurineMedicalDoctor
- ToyFigurineHeadOfPersonnel
- ToyFigurineBooCat
Copy link
Contributor

@Shegare Shegare Dec 10, 2024

Choose a reason for hiding this comment

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

Добавь в конце строки - # Stories

@@ -101,7 +101,7 @@
- ToyFigurineLawyer
- ToyFigurineCargoTech
- ToyFigurineSalvage
- ToyFigurineQuartermaster
- ToyFigurineQuartermaster
Copy link
Contributor

@Shegare Shegare Dec 10, 2024

Choose a reason for hiding this comment

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

Добавь в конце строки - # Stories

Comment on lines 7 to 12
- type: localizedDataset
id: FigurinesBooCat
values:
prefix: figurines-boo-cat-
count: 4

Copy link
Contributor

Choose a reason for hiding this comment

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

Это надо в Stories папку

Comment on lines 5 to 6
ent-ToyFigurineBooCat = фигурка зловещего кота
.desc = Фигурка, изображающая зловещего кота, который обычно обитает в темноте технических туннелей.
Copy link
Contributor

Choose a reason for hiding this comment

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

Это надо в _stories папку

Comment on lines 3 to 6
figurines-boo-cat-1 = Бу! Испугался? Не бойся, я друг, я тебя не обижу.
figurines-boo-cat-2 = Иди сюда, иди ко мне, посмотри мне в глаза.
figurines-boo-cat-3 = Ты видишь меня? Я тоже тебя вижу.
figurines-boo-cat-4 = Ты не хочешь? Почему? Что-то не так?
Copy link
Contributor

Choose a reason for hiding this comment

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

Это надо в _stories папку

Comment on lines 4 to 8
figurines-boo-cat-1 = Boo! Are you scared? Don't be afraid, I'm a friend, I won't hurt you.
figurines-boo-cat-2 = Come here, come to me, sit next to me, look into my eyes.
figurines-boo-cat-3 = Do you see me? I can see you too.
figurines-boo-cat-4 = Don't you want to? Why? Is something wrong?

Copy link
Contributor

Choose a reason for hiding this comment

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

Это надо в _stories папку или просто удалить

@Shegare
Copy link
Contributor

Shegare commented Dec 10, 2024

@PCCartem нам важно понимать какая часть оффовская, а какая наша, чтобы не ломать голову при апстримах

Comment on lines 165 to 167
},
{
"name": "boo_cat"
Copy link
Contributor

Choose a reason for hiding this comment

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

Это надо в Stories папку

@PCCartem
Copy link
Author

@Shegare Мне всё понятно, займусь по возможности и перенесу.
Спасибо за ревью.

@PCCartem
Copy link
Author

PCCartem commented Dec 11, 2024

Пришлось добавить компонент Speech и UseDelay, и в саму фигурку SpeakOnUse.
Теперь все фигурки могут говорить, по идее, если дадите список фраз для того же тенеморфа, можно и их сделать интерактивными, если компонент SpeakOnUse не прописан, как и раньше, при нажатии ничего не произойдёт, потестировал.

Там есть кое-где вот такого типа:

name: фигурка инквизитора
description: Я пришел за тобой!

Можно добавить, но в задачу это не входило, поэтому не стал трогать, да и контекста некоторых фигурок просто не знаю.

@MetalSage
Copy link
Owner

Отказано, не вижу смысл добавлять очередной мем в нашу сборку, который уже неактуален.

@MetalSage MetalSage closed this Dec 14, 2024
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.

3 participants