-
Notifications
You must be signed in to change notification settings - Fork 444
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
Fixes light attachments setting and add pref for ammo counter #6251
Conversation
I don't think ammo counter change is needed unless you make it a setting. |
Sure thing. |
You can toggle it now. |
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.
In the future fill out the PR template; don't just omit it.
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.
As mentioned on discord:
- This would be better as a player preference rather than something you need to set per weapon IMO
- I have no idea where verb desc even gets displayed for us, so conveying the < 15 bullet information needs to also be somewhere else; or just make it a different option for the pref (e.g. every bullet, every 5 bullets unless low, every 5 bullets)
- Additionally, you should mention this only applies to semi fire (e.g. make the preference called semi-auto ammo counter - because of note this doesn't work for automatic but I think that's fine)
Co-authored-by: Drathek <76988376+Drulikar@users.noreply.github.com>
Co-authored-by: Drathek <76988376+Drulikar@users.noreply.github.com>
Co-authored-by: Drathek <76988376+Drulikar@users.noreply.github.com>
About the pull request
Closes #6238, also made the ammo display less spammy.
🆑
qol: Toggleable ammo counter, can now cycle between displaying every fifth or single bullet.
fix: Light attachments will no longer automatically turn on when the weapon is equipped from suit storage
/:cl: