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

Removes Squad Optics as Default Helmet Visors #5583

Closed
wants to merge 4 commits into from

Conversation

Staykeu
Copy link
Contributor

@Staykeu Staykeu commented Jan 28, 2024

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:

@github-actions github-actions bot added the Quality of Life Make the game harder to play label Jan 28, 2024
Copy link
Contributor

@Drulikar Drulikar left a 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.

@Drulikar Drulikar marked this pull request as draft January 30, 2024 07:56
@Wintermote
Copy link

Does this also change the current functionality so squad optics can be removed with a screwdriver after install?

@Staykeu
Copy link
Contributor Author

Staykeu commented Jan 30, 2024

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.

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.

@Staykeu
Copy link
Contributor Author

Staykeu commented Jan 30, 2024

Does this also change the current functionality so squad optics can be removed with a screwdriver after install?

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.

@Staykeu Staykeu marked this pull request as ready for review January 30, 2024 15:32
@Staykeu Staykeu requested a review from Drulikar January 30, 2024 17:36
@harryob
Copy link
Member

harryob commented Jan 31, 2024

I would argue this is unnecessary, all of it.

not how requested changes work

@harryob harryob closed this Jan 31, 2024
@Staykeu
Copy link
Contributor Author

Staykeu commented Jan 31, 2024

I would argue this is unnecessary, all of it.

not how requested changes work

so you just fucking close it? awesome, amazing

@Drulikar
Copy link
Contributor

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).

@Staykeu
Copy link
Contributor Author

Staykeu commented Jan 31, 2024

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

@Staykeu Staykeu deleted the squad-optic branch September 13, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Quality of Life Make the game harder to play
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants