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 another bell clothing item #762

Merged
merged 7 commits into from
Dec 26, 2023

Conversation

Mnemotechnician
Copy link
Contributor

@Mnemotechnician Mnemotechnician commented Dec 25, 2023

About the PR

So recently I discovered jester shoes and I loved the sound they make. But there was one problem: they don't allow you to wear magboots or some other kind of shoes at the same time. So I had the idea to add a new item that would make the bell sound and make it use the neck slot (since it's only used for vanity items), and someone suggested a bell collar. And now here we are!

The item emits the same sound as jester shoes when it moves, or when its wearer moves. It doesn't emit any sound when placed and moved a container, or when its wearer is placed and moved inside a container.

Why / Balance

Monke hears jingling sound -> monke feels happy. Definitely not useful if you plan on sneaking on someone.

Technical details

Added an EmitsSoundOnMove component and the accompanying system.

Media

https://cdn.discordapp.com/attachments/1123826879892299783/1188964506353807591/weeee-2023-12-26_00.58.46.mp4?ex=659c707c&is=6589fb7c&hm=eed6998eff2a563821c05bba039d7f7a2591b66f33209148f8e922084f231777&

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

None

Changelog
🆑

  • add: Added a new wearable bell item to the theater vend.

Copy link
Contributor

github-actions bot commented Dec 25, 2023

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

Resources/Textures/_NF/Clothing/Neck/bellcollar.rsi

State Old New Status
equipped-NECK Added
icon Added
inhand-left Added
inhand-right Added

Edit: diff updated after b2667dc

Copy link
Contributor

@FoxxoTrystan FoxxoTrystan left a comment

Choose a reason for hiding this comment

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

This is cursed, your giving vulps a collar! were not dogs!

Anywho good job, tho is the bellcollar.rsi a full sprite of your work or is it a modified version of an actual collar file?

@Mnemotechnician
Copy link
Contributor Author

Anywho good job, tho is the bellcollar.rsi a full sprite of your work or is it a modified version of an actual collar file?

Mine, made in gimp using some other clothing piece as a template for the rsi

And oooooops I gotta fix it making the sound while in a pocket-

@Cheackraze Cheackraze merged commit 0519335 into new-frontiers-14:master Dec 26, 2023
10 of 11 checks passed
FrontierATC added a commit that referenced this pull request Dec 26, 2023
@Mnemotechnician Mnemotechnician deleted the feat/bell branch December 26, 2023 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants