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

Pocket Knife - Hephaestus Soul #20081

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

Conversation

courierbravo
Copy link
Contributor

Adds a new pocket knife and some alternative versions.
Adds a pocket butter knife, a unique item restricted to Hephaestus crew.

@BotBOREALIS BotBOREALIS added the Sprites Adds new or changes existing sprites. label Oct 18, 2024
@courierbravo
Copy link
Contributor Author

!review

Copy link
Contributor

@FluffyGhoster FluffyGhoster left a comment

Choose a reason for hiding this comment

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

In addition to the line notes, the knife shouldn't do the same damage as closed as it does as open, you need to change the damage values depending on its status

code/game/objects/items/weapons/material/kitchen.dm Outdated Show resolved Hide resolved
code/game/objects/items/weapons/material/kitchen.dm Outdated Show resolved Hide resolved
code/game/objects/items/weapons/material/kitchen.dm Outdated Show resolved Hide resolved
code/game/objects/items/weapons/material/kitchen.dm Outdated Show resolved Hide resolved
code/game/objects/items/weapons/material/kitchen.dm Outdated Show resolved Hide resolved
code/game/objects/items/weapons/material/kitchen.dm Outdated Show resolved Hide resolved
@courierbravo
Copy link
Contributor Author

Resolving issues thanks to Fluffy making my code much nicer.

Comment on lines +359 to +362
/obj/item/toy/balloon/syndicate = 4,
/obj/item/material/knife/butterfly/switchblade/pocket_knife = 2.5,
/obj/item/material/knife/butterfly/switchblade/pocket_knife/blue = 2.5,
/obj/item/material/knife/butterfly/switchblade/pocket_knife/red = 2.5
Copy link
Contributor

Choose a reason for hiding this comment

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

Spawn chance should be much lower and should be shared with normal butterfly knives.

Not only this has a higher chance of rolling, it's also rolling three times

Comment on lines +204 to +208
/obj/item/material/knife/butterfly/switchblade/pocket_knife/blue
icon_state = "pocketblue"

/obj/item/material/knife/butterfly/switchblade/pocket_knife/red
icon_state = "pocketred"
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't manually create colored variants, use recolorable code (see : screwdrivers)

Copy link
Contributor

@alsoandanswer alsoandanswer 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 missing inhands

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Review Required Sprites Adds new or changes existing sprites.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants