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

added status msg to oil splatter knockdown #47850

Conversation

Jamuro-g
Copy link
Contributor

@Jamuro-g Jamuro-g commented Mar 3, 2021

Summary

Bugfixes "Added status msg to oil splatter field knockdown"

Purpose of change

Slipping on mechanical fluid fields gave the same status msg to the player as any other knockdown, potentially causing confusion.

Resolves #47847

Describe the solution

Made the knockdown effect of oil fields it's own effect and changed the status message sent to the player

Testing

  • Spawn in a crow II and shoot it
  • Move over the spawned mechanical splatter fields until you get knocked down

@Fris0uman
Copy link
Contributor

You need to format your json change https://github.com/CleverRaven/Cataclysm-DDA/blob/master/doc/JSON_STYLE.md
You can use this tool http://dev.narc.ro/cataclysm/format.html by just copy pasting stuf in it.

Did you check that this downed_oil effect actually does something other than just displaying the message?

@Jamuro-g Jamuro-g marked this pull request as draft March 3, 2021 14:25
@Qrox
Copy link
Contributor

Qrox commented Mar 3, 2021

Did you check that this downed_oil effect actually does something other than just displaying the message?

In addition to that, this also introduces the doubtful situation where you can be "knocked to the floor" by monster attack when you are already on the ground after slipping.

I suggest just changing the downed effect message to something more general like "you drop to the ground" and print an additional message when slipping on oil.

@jagoly
Copy link
Contributor

jagoly commented Mar 3, 2021

Why not just print an extra message before applying the effect?

"You slip on the mechanical fluid!"
"You're knocked to floor!"

I'd argue this sounds perfectly reasonable. It reads as a cause, then an effect, like the other common causes for getting knocked to the floor (monster hits your torso, etc).

@Fris0uman
Copy link
Contributor

Hardcoding one more message is not great though.

@Jamuro-g
Copy link
Contributor Author

Jamuro-g commented Mar 3, 2021

oh wow this got more attention than i thought ... it wasn't exactly ready (was just a bit slow on hitting the draft button)

anyways ... if we don't want to use a sepperate effect for this then i can just add the message, no problem.

In addition to that, this also introduces the doubtful situation where you can be "knocked to the floor" by monster attack when you are already on the ground after slipping.

This shouldn't happen, the downed_oil effect is added to the dragdown check.

@Jamuro-g
Copy link
Contributor Author

Jamuro-g commented Mar 5, 2021

oil splatter no longer causes a downed effect, obsoleting this pr (closed)

@Jamuro-g Jamuro-g closed this Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[JSON] Changes (can be) made in JSON
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Slipping on mechanical fluid should have it's own message
5 participants