Skip to content

Auto Magazine Ejection Works for Irremovable Attachments #18067

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

Merged
merged 6 commits into from
Jul 18, 2025

Conversation

Runian
Copy link
Contributor

@Runian Runian commented Jul 8, 2025

About The Pull Request

The verb "Toggle Automatic Magazine Ejection (Weapon)" now gives a choice of which weapon and/or attachment to toggle if there is more than one possible item that can make use of it.

Why It's Good For The Game

The ability to toggle auto-eject mags is locked behind having to actually hold the attachment in your hand. Some attachments are irremovable which makes it impossible to normally toggle automatic magazine ejection. This fixes that.

Changelog

🆑
qol: Toggle Automatic Magazine Ejection (Weapon) now lets you choose between your weapon and the weapon's attachments.
/:cl:

@github-actions github-actions bot added the Quality of Life Minor changes that make the game nicer to play label Jul 8, 2025
Copy link
Contributor

@ivanmixo ivanmixo left a comment

Choose a reason for hiding this comment

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

Surely you can come up with a better way

@Runian
Copy link
Contributor Author

Runian commented Jul 8, 2025

I guess the other option would be to copy-paste the verb and only allow it to work if the underbarrel attachment can make use of it.

@Lumipharon
Copy link
Contributor

yeah don't do it this way. Make the toggle verb check if the underbarrel is even valid for autoeject or not (presumably its just this, plasma pistol and tx54), then give you a list so you can choose which one you are actually trying to toggle.

@Runian Runian changed the title SG-153 can be deattached Auto Magazine Ejection Works for Irremovable Attachments Jul 8, 2025
@Runian Runian requested a review from ivanmixo July 8, 2025 16:34
@TiviPlus TiviPlus merged commit 1c3e8d0 into tgstation:master Jul 18, 2025
42 checks passed
github-actions bot added a commit that referenced this pull request Jul 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Quality of Life Minor changes that make the game nicer to play
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants