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

M56 Headset Functionality #454

Closed
wants to merge 35 commits into from
Closed

Conversation

Max-023
Copy link
Contributor

@Max-023 Max-023 commented Sep 14, 2024

About the pull request

Makes it so the M56 headset is required for IFF functionality as well as reciprocates the existence of a sprited camera with code that allows it to replace the need for a helmet.

Additionally, adds the custom smartgunner machete scabbard to replace the regular large machete.

Explain why it's good for the game

This doesn't fix the issue with the headset's awkward existence, but at least gives it some purpose to be used that is lore friendly. The working camera also means it can take the heat off the backs of smartgunners when commanders try to make sure they can see the cameras of their platoon, making Drakemaxxing legal.

As for the scabbard looks a lot better with the smartgunner harness and gun when it is all stowed, as the regular scabbard had clipping issues.

Changelog

🆑
add: M5 Camera Gear as a generic overwatch camera item
add: Firing with IFF now requires an M56 headset
add: The M56 headset now has camera code
add: Smartgunner machete scabbard added and placed in SG lockers
/:cl:

Dumbgunner added 30 commits March 27, 2024 15:58
This reverts commit b1f8fc5.
@Max-023 Max-023 marked this pull request as ready for review September 14, 2024 23:51
@@ -23,7 +23,7 @@
/obj/item/device/walkman,
)
var/list/smartgun_back = list(
/obj/item/storage/large_holster/machete,
/obj/item/storage/large_holster/machete/smartgunner,
Copy link
Contributor

Choose a reason for hiding this comment

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

some smartgunners use loadout machetes and that would prevent them from using it, i see no reason for this change

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is added to the locker as well so they would still be able to use the machete.

@Max-023 Max-023 closed this Sep 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants