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

Everybody Needs a Kiss #179

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

Conversation

neuPanda
Copy link
Contributor

@neuPanda neuPanda commented Sep 7, 2024

Description.
added kiss interaction and texts
added lick interaction and texts

🆑

  • add: Kiss an Lick

@github-actions github-actions bot added Status: Needs Review Someone please review this Changes: Localization Changes any ftl files Changes: YML Changes any yml files labels Sep 7, 2024
@Floof-Station-Bot Floof-Station-Bot changed the title Everybody needs a kiss Everybody Needs a Kiss Sep 7, 2024
Copy link
Contributor

@Mnemotechnician Mnemotechnician left a comment

Choose a reason for hiding this comment

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

Same applies to the second action-

hideByRequirement: true
requirement:
!type:MobStateRequirement
inverted: true
Copy link
Contributor

Choose a reason for hiding this comment

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

You don't want this to be usable on dead bodies. Probably.

Suggested change
inverted: true
allowedStates: [Alive, Critical]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

:( no, there are people who are going to want that

@Mnemotechnician
Copy link
Contributor

Simple-Station/Einstein-Engines#823 has been merged and is going to be included in #178, so uh please update those later to add some kinda moodlet.

@Fansana
Copy link
Owner

Fansana commented Sep 10, 2024

#178 is added please revisit this

@Mnemotechnician Mnemotechnician mentioned this pull request Sep 13, 2024
@FoxxoTrystan FoxxoTrystan added the Priority: 2-Medium Needs to be resolved at some point label Sep 14, 2024
@neuPanda
Copy link
Contributor Author

i fuckin hate yml

@Memeji
Copy link
Collaborator

Memeji commented Sep 14, 2024

What is the current status for this PR?

Copy link
Contributor

@ShatteredSwords ShatteredSwords left a comment

Choose a reason for hiding this comment

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

The kiss and lick make a hug sound. If this is a limitation or if it is hard to find these, that is totally fair. If you think you can find a properly licensed sound for each of these actions though, that'd be cool!

@neuPanda
Copy link
Contributor Author

The kiss and lick make a hug sound. If this is a limitation or if it is hard to find these, that is totally fair. If you think you can find a properly licensed sound for each of these actions though, that'd be cool!

not sure where the sounds are or how they work, but i am sure there is something out there.
that said you also got me to go back and look at it and realize i missed a 2 lines in the copy paste that needed to be updated

@ShatteredSwords
Copy link
Contributor

not sure where the sounds are or how they work, but i am sure there is something out there. that said you also got me to go back and look at it and realize i missed a 2 lines in the copy paste that needed to be updated

Base interactions are located here, which is what looks like ties the sound to whatever. Looks like you'd have to make a new base interaction within the Floof folder and parent it to that. Completely up to you of course! I think we could do it in the future just as easily!

@Mnemotechnician
Copy link
Contributor

There's no need to create new bases unless you intend to re-use them

@neuPanda
Copy link
Contributor Author

I need to run everything again after the changes i did yesterday just have not had a chance to open VS yet. it was just an issue i saw in the PR, so i did the quick fix remotely and do still intend to test it

@Memeji
Copy link
Collaborator

Memeji commented Sep 19, 2024

I need to run everything again after the changes i did yesterday just have not had a chance to open VS yet. it was just an issue i saw in the PR, so i did the quick fix remotely and do still intend to test it

Any update on this?

@neuPanda
Copy link
Contributor Author

I need to run everything again after the changes i did yesterday just have not had a chance to open VS yet. it was just an issue i saw in the PR, so i did the quick fix remotely and do still intend to test it

Any update on this?

sorry not yet, been busy with work

@github-actions github-actions bot added the Changes: Audio Changes any ogg files label Sep 19, 2024
@neuPanda
Copy link
Contributor Author

audio's have been updated

@ShatteredSwords
Copy link
Contributor

I will test this when I have the chance, very excited!

Copy link
Contributor

@ShatteredSwords ShatteredSwords left a comment

Choose a reason for hiding this comment

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

Sounds don't seem to be working, still uses the default sounds after playtesting. Not sure why! Everything else still is good though.

@@ -0,0 +1,29 @@
# Base interaction that involves hands
Copy link
Contributor

Choose a reason for hiding this comment

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

This entire file can be removed in favor of the suggestion below

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will look into this when i have some time, been a bit busty and have not had time to look into this yet. sorry for the delay

Comment on lines +3 to +10
parent: [BaseGlobal, BaseLips]
priority: 0
delay: 0.4
range: {max: 1}
hideByRequirement: true
requirement:
!type:MobStateRequirement
inverted: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Lesser priority, inlined effect

Suggested change
parent: [BaseGlobal, BaseLips]
priority: 0
delay: 0.4
range: {max: 1}
hideByRequirement: true
requirement:
!type:MobStateRequirement
inverted: true
parent: [BaseGlobal, BaseHands]
priority: -1
delay: 0.4
range: {max: 1}
hideByRequirement: true
requirement:
!type:MobStateRequirement
inverted: true
effectSuccess:
popup: Obvious
sound: {path: /Audio/Floof/Lewd/kiss.ogg}

Comment on lines +18 to +26
parent: [BaseGlobal, BaseTongue]
priority: 0
#icon: /Textures/Interface/Actions/hug.png
delay: 0.7
range: {max: 1}
hideByRequirement: true
requirement:
!type:MobStateRequirement
inverted: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Similarly,

Suggested change
parent: [BaseGlobal, BaseTongue]
priority: 0
#icon: /Textures/Interface/Actions/hug.png
delay: 0.7
range: {max: 1}
hideByRequirement: true
requirement:
!type:MobStateRequirement
inverted: true
parent: [BaseGlobal, BaseTongue]
priority: -2
#icon: /Textures/Interface/Actions/hug.png
delay: 0.7
range: {max: 1}
hideByRequirement: true
effectSuccess:
popup: Obvious
sound: {path: /Audio/Floof/Lewd/lick.ogg}
requirement:
!type:MobStateRequirement
inverted: true

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Audio Changes any ogg files Changes: Localization Changes any ftl files Changes: YML Changes any yml files Priority: 2-Medium Needs to be resolved at some point Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants