-
Notifications
You must be signed in to change notification settings - Fork 566
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
Removes Squad Optics as Default Helmet Visors #5583
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The visor does work - however headsets currently also provide this HUD for USCM so it is currently redundant assuming you have a headset.
I am half tempted to just remove the HUD from headsets however for now in this PR please instead simply address the reason why you were making this PR in the first place: Revert your current changes and instead if you already have a HUD, have the visor toggle skip that visor.
This means it should also skip a medhud visor if you have an eye item giving the medical hud. Perform the check by looking at the the data in mob_hud.dm.
Does this also change the current functionality so squad optics can be removed with a screwdriver after install? |
I would argue this is unnecessary, all of it. The squad HUD is still available in squad prep's vending, and while the headset is in its current state (where it should stay), there is no reason to have a squad HUD. |
Currently, the squad optic is built into the helmet, built-in visors cannot be removed. Attached visors CAN be removed, so if you attach a squad visor, it can be removed. |
not how requested changes work |
so you just fucking close it? awesome, amazing |
Feel free to open another PR with the requested change. It both keeps the functional visor in the helmet, and resolves your issue needing to perform an extra skip over a hud you already have from your headset, AND would do the same for medhuds too should someone also be wearing one (e.g. prescription maybe). |
#5600 harry helped me out with it, this is the new pr with the changes you requested |
About the pull request
Removes the useless "squad optic" as a default attachment for marine helmets.
Explain why it's good for the game
The squad optic is functionally useless and only serves as a cosmetic item, most players don't use the squad optic and most of the time it only serves as another optic you have to flick through to get to the optic you want. Those who still want to use the squad optic can get them from squad prep rooms, it's not fully removed.
Testing Photographs and Procedure
Screenshots & Videos
Put screenshots and videos here with an empty line between the screenshots and the
<details>
tags.Changelog
🆑 Stakeyng
qol: Removes the need to cycle through the squad optic to get to other optics (welding/medical/nv).
/:cl: