-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Ammo boxes now have sprites for being parially filled! #32930
Ammo boxes now have sprites for being parially filled! #32930
Conversation
Created new sprites for all the partial fills and modified the ammo-case ymls to accomodate the new stages
RSI Diff Bot; head commit a4b4c03 merging into c221ef0 Resources/Textures/Objects/Weapons/Guns/Ammunition/Boxes/anti_materiel.rsi
Resources/Textures/Objects/Weapons/Guns/Ammunition/Boxes/caseless_rifle.rsi
Resources/Textures/Objects/Weapons/Guns/Ammunition/Boxes/light_rifle.rsi
Resources/Textures/Objects/Weapons/Guns/Ammunition/Boxes/magnum.rsi
|
Test has again failed for reasons seemingly unrelated to the changes made and completely related to the test being consistently buggy and intermittently broken. |
This is nice, thank you for your contribution |
As an aside, I found it weird that some (light_rifle.rsi, magnum.rsi, caseless_rifle.rsi) of these have very clearly "4 quadrants of fill" but go from 1 missing to 3 missing. But it seems this has nothing to do with the configuration in this PR, there isn't even a "2 missing" sprite for those boxes. |
if ss13 had partial sprites these aren't from there, I made them myself. As for why, it was kind of stylistic choice and kind of me being lazy and not wanting to make more sprites |
danke |
About the PR
Ammo boxes no longer show up as empty after removing just one bullet.
Why / Balance
Annoying and ugly.
Technical details
Added new sprites, modified ammo-box ymls to use the new stages
Media
Requirements
Breaking changes
None (probably)
Changelog
🆑