Skip to content

feat: port periscope#953

Merged
MaxCWhitehead merged 2 commits intofishfolk:mainfrom
DRuppFv:periscope_port
Apr 1, 2024
Merged

feat: port periscope#953
MaxCWhitehead merged 2 commits intofishfolk:mainfrom
DRuppFv:periscope_port

Conversation

@DRuppFv
Copy link
Contributor

@DRuppFv DRuppFv commented Mar 30, 2024

Periscope ported, this closes #151.

It's similar to the musket and is mainly useful if the enemy is higher than you. Used the artwork from #151 but the concept art confused me a bit as it says to be "similar to the musket" but in the art it looks like more a shotgun as it has some bullet spread, but adding a very low bullet spread would be a nice idea here so let me know if you think it should be used.

As it's very similar to the musket, it had a certain amount of code duplication, which I think we could discuss together and come to a decision whether to generalize something elsewhere or whether we want to make adjustments to add more uniqueness to the designs.

shooting_periscope

periscope

@erlend-sh
Copy link
Member

erlend-sh commented Mar 31, 2024

As it's very similar to the musket, it had a certain amount of code duplication

That could also make it a good candidate for a Lua implementation. This PR is good as-is, just putting a note in for later.

@MaxCWhitehead
Copy link
Collaborator

Awesome!

@MaxCWhitehead MaxCWhitehead merged commit 147547f into fishfolk:main Apr 1, 2024
@DRuppFv DRuppFv deleted the periscope_port branch April 4, 2024 04:48
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.

[SPEC] Periscope Rifle

3 participants