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

Weapon resizing #34387

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Weapon resizing #34387

wants to merge 10 commits into from

Conversation

Nox38
Copy link
Contributor

@Nox38 Nox38 commented Jan 11, 2025

About the PR

Added custom dimensions to every firearm, for better realism. Slightly nerfs the satchel too, by making it less able to carry a full-length assault rifle.

Why / Balance

This started as simply "why isn't the Enforcer 1x5?" and was quickly followed by other strange questions like "why is the assault rifle a square?". This has led me to change every single gun to have a somewhat more realistic and balanced inventory size.

Technical details

Custom dimensions were added to the item component of every firearm.

Media

image
image

Requirements

Changelog
🆑 Nox38

  • tweak: The compact SMGs (C20r, WT550, Atredies, Disabler SMG) are now 2x3.
  • tweak: The heavier pump shotguns (Enforcer & Kammerer) are 2x5.
  • tweak: The China Lake and L6 are big at 3x6. Your back slot is suggested but not technically required.
  • tweak: The assault rifles take up a bit less space at 3x5.
  • tweak: The Hristov folds up to just 3x5 to fit into a briefcase. The 28-inch barrel probably splits in half or something for this, idk.
  • tweak: Energy carbines (Laser & pulse) are much more portable at 2x5.
  • tweak: Energy rifles (X-ray, Laser cannon, Pulse) are somewhat more portable at 2x6.
  • tweak: The advanced laser pistol and antique laser pistol are now 2x2 to fit their models (Both can fit in your pocket now!).
  • tweak: All pistol-gripped pistols are now pistol-shaped (like the svalinn).
  • tweak: Renamed 'laser rifle' to 'laser carbine' to better match its own ID and it's smaller size.
  • tweak: Rotated retro laser pistol's sprite to actually fit the pistol shape (thanks ItzChilde!)
  • tweak: RPG-7 and handheld cannon are now ginormous. That is not going in your bag.
    -->

@github-actions github-actions bot added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. size/M Denotes a PR that changes 100-999 lines. labels Jan 11, 2025
@Nox38 Nox38 marked this pull request as ready for review January 12, 2025 01:19
@kosticia
Copy link
Contributor

Great PR! But in my opinion satchels will be so much useless for antagonists and security, because biggest part of guns will be so much big for them.

@Pangogie Pangogie added T: Balance Change Type: Balance changes through direct value changes, or changes to mechanics that affect it A: General Interactions Area: General in-game interactions that don't relate to another area. size/S Denotes a PR that changes 10-99 lines. and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/S Denotes a PR that changes 10-99 lines. labels Jan 12, 2025
@Nox38
Copy link
Contributor Author

Nox38 commented Jan 12, 2025

Great PR! But in my opinion satchels will be so much useless for antagonists and security, because biggest part of guns will be so much big for them.

I'm not sure what guns would still work as 4 long besides the SMGs and the bulldog. The goal isn't to make the satchel useless for those roles, but make satchel vs backpack more of a serious question than satchel being better always. If you have any change suggestions let me know!

@IProduceWidgets
Copy link
Contributor

IProduceWidgets commented Jan 13, 2025

You'll have to double check it works because I remember there being issues with some visualizers & storage sprite changes in the past, but the pistols would look a lot better offset.

public Vector2i StoredOffset;

Similarly, the mosin and hristov probably need non-diagonal sprites instead of just making the inv size bigger

@Nox38
Copy link
Contributor Author

Nox38 commented Jan 13, 2025

You'll have to double check it works because I remember there being issues with some visualizers & storage sprite changes in the past, but the pistols would look a lot better offset.

Ive been wanting to do this, but don't know where would I find/change each item's offset?

Similarly, the mosin and hristov probably need non-diagonal sprites instead of just making the inv size bigger

unfortunately I do not know how to make/edit sprites, but I like the idea.
The Hristov isn't a huge deal in my opinion because you'd need to disassemble it to fit it in a bag in the first place, explaining the square-er shape, but the Mosin could definitely use a horizontal sprite.

@LaCumbiaDelCoronavirus
Copy link

I feel like the enforcer, bulldog and Drozd should keep their original size, as they took large amounts of space in bags to account for how good they are as weapons by themselves. The SMGs (other than drozd, but especially C20s) should really also be longer so that you can't fit, what, 6 of them in one satchel, let alone a bag.
Other than that it looks like a nice PR!

@K-Dynamic
Copy link
Contributor

I feel like the point of gridinv was to standardise sizes, this is going in the opposite direction

@ArtisticRoomba
Copy link
Contributor

I feel like the point of gridinv was to standardise sizes, this is going in the opposite direction

The reason why GridInv has the same sizes for a lot of items is actually because they're inherited from the default sizes of small, medium, large, etc. Expanding the sizes into more unique shapes encourages creativity and funny tetris inventory management which is more fun IMO.

image
image

@pheenty
Copy link
Contributor

pheenty commented Jan 13, 2025

Syndie shoulder holster should definitely be changed somehow if this gets merged, something like a literal slot for a gun would work (like the jackboots have a slot for a pistol)

@github-actions github-actions bot added the Changes: Sprites Changes: Might require knowledge of spriting or visual design. label Jan 13, 2025
Copy link
Contributor

RSI Diff Bot; head commit 1d2833c merging into e271a5a
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Objects/Weapons/Guns/Battery/laser_retro.rsi

State Old New Status
base Modified
mag-unshaded-0 Modified
mag-unshaded-1 Modified
mag-unshaded-2 Modified
mag-unshaded-3 Modified
mag-unshaded-4 Modified

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: General Interactions Area: General in-game interactions that don't relate to another area. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. Changes: Sprites Changes: Might require knowledge of spriting or visual design. size/M Denotes a PR that changes 100-999 lines. T: Balance Change Type: Balance changes through direct value changes, or changes to mechanics that affect it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants