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

Added bags for HoP #2856

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

Conversation

Possumongus
Copy link

About the PR

I've decided to add specific clothing to the HoP, mainly bags. I'm planning on also doing this for DeltaV specific roles, but I figured this would be a start
They are available in the loadout screen, and you can put em on at the start of the game

Why / Balance

I felt that command shouldn't be wearing gray bags, one is for style reasons. It breaks up the visual consistency of the character and undermines certain command roles to feel less important. I also feel its vital that command roles have easy to distinguish bags. It lets security know when someone has something they shouldn't, and it paints a big X for the antags to go after.

About the sprites themselves, I've tried to breakup the shape a little bit while still keeping it recognizable, I've tested the sprites in B/W etc to make sure they are visually distinguishable from other bags. I've added small extra bags, changed the shape in general just to make it more visually distinct. I might update the sprites and any and all feedback is well appreciated.

Technical details

Changed the load-out group of the HoP in the main prototype file, added comments to keep it easy to see what I changed. Rest is just adding the items themselves and the associated files to make the sprites work.

Media

Media attached
imasdasdasdsaage

imadsadasdasdge

Requirements

  • [ X ] I have tested all added content and changes.
  • [ X ] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

only changed 1 baseline file which was the load-out group for HoP's backpacks. Commented original bags for easily delete and fix in case they need to changed
Changelog

🆑

  • add: Added new HoP backpacks to keep visual consistency!
  • remove: Removed default bags from the hop loadout

@Possumongus Possumongus requested review from a team as code owners February 2, 2025 16:04
@github-actions github-actions bot added S: Needs Review size/M 64-255 lines Changes: YML Changes any yml files Changes: Sprite Changes any png or json in an rsi and removed size/M 64-255 lines labels Feb 2, 2025
@github-actions github-actions bot added the size/M 64-255 lines label Feb 2, 2025
Copy link
Contributor

github-actions bot commented Feb 2, 2025

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

Resources/Textures/_DV/Clothing/Back/Backpacks/hop.rsi

State Old New Status
equipped-BACKPACK Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/_DV/Clothing/Back/Duffels/hop.rsi

State Old New Status
equipped-BACKPACK Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/_DV/Clothing/Back/Satchels/hop.rsi

State Old New Status
equipped-BACKPACK Added
icon Added
inhand-left Added
inhand-right Added

Edit: diff updated after dbe1c88

@Possumongus Possumongus changed the title mogus Added bags for HoP Feb 2, 2025
Copy link
Contributor

@Radezolid Radezolid left a comment

Choose a reason for hiding this comment

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

All the JSON needs proper indentation (4 spaces instead of 2), for an example check this one:

{
"version": 1,
"license": "CC0-1.0",
"copyright": "Original work by TJohnson.",
"size": {
"x": 32,
"y": 32
},
"states": [
{
"name": "icon"
},
{
"name": "equipped-INNERCLOTHING",
"directions": 4
},
{
"name": "inhand-left",
"directions": 4
},
{
"name": "inhand-right",
"directions": 4
},
{
"name": "rolled-equipped-INNERCLOTHING",
"directions": 4
}
]
}

@@ -0,0 +1,33 @@
- type: entity
parent: [Clothing, ContentsExplosionResistanceBase]
Copy link
Contributor

Choose a reason for hiding this comment

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

Inherit from ClothingBackpack

Comment on lines 10 to 32
- type: Item
size: Huge
- type: Clothing
quickEquip: false
slots:
- back
- type: Storage
grid:
- 0,0,6,3
maxItemSize: Huge
- type: ContainerContainer
containers:
storagebase: !type:Container
ents: []
- type: UserInterface
interfaces:
enum.StorageUiKey.Key:
type: StorageBoundUserInterface
# to prevent bag open/honk spam
- type: UseDelay
delay: 0.5
- type: ExplosionResistance
damageCoefficient: 0.9
Copy link
Contributor

Choose a reason for hiding this comment

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

Delete once you do the correct inheritance.

@@ -0,0 +1,16 @@
- type: entity
parent: ClothingBackpack
Copy link
Contributor

Choose a reason for hiding this comment

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

Inherit from ClothingBackpackDuffel.

Comment on lines 9 to 16
- type: Storage
maxItemSize: Huge
grid:
- 0,0,7,4
- type: ClothingSpeedModifier
walkModifier: 1
sprintModifier: 0.9
- type: HeldSpeedModifier
Copy link
Contributor

Choose a reason for hiding this comment

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

Delete once the correct prototype is inherited.

@@ -0,0 +1,13 @@
- type: entity
parent: ClothingBackpack
Copy link
Contributor

Choose a reason for hiding this comment

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

Inherit from ClothingBackpackSatchel

Comment on lines 9 to 13
- type: Storage
grid:
- 0,0,1,3
- 3,0,6,3
- 8,0,9,3
Copy link
Contributor

Choose a reason for hiding this comment

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

Delete once the correct protoype is inherited.

Comment on lines 761 to 762


Copy link
Contributor

Choose a reason for hiding this comment

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

untouch

Copy link
Contributor

Choose a reason for hiding this comment

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

Needs proper indentation of four spaces instead of two.

Copy link
Contributor

Choose a reason for hiding this comment

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

Needs proper indentation of four spaces instead of two.

Copy link
Contributor

Choose a reason for hiding this comment

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

Needs proper indentation of four spaces instead of two.

Copy link
Contributor

@Radezolid Radezolid left a comment

Choose a reason for hiding this comment

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

Forgot to mention these.

Comment on lines 181 to 183
## - CommonBackpack
## - CommonSatchel
## - CommonDuffel
Copy link
Contributor

Choose a reason for hiding this comment

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

Forgot to mention, these should be properly commented out, remove one character of these two ##

Comment on lines 181 to 186
## - CommonBackpack
## - CommonSatchel
## - CommonDuffel
- BackpackHoP #DELTA V
- SatchelHoP #DELTA V
- HopDuffel #DELTA V
Copy link
Contributor

Choose a reason for hiding this comment

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

All of this should be enclosed in a multi line DeltaV comment, like this:

# Start DeltaV changes
blah
blah
# End of DeltaV changes

Copy link
Author

Choose a reason for hiding this comment

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

Fixed, I think

@ThataKat
Copy link
Contributor

ThataKat commented Feb 2, 2025

Under direction review, get back to you in a day or so

Lyndomen
Lyndomen previously approved these changes Feb 2, 2025
@Possumongus
Copy link
Author

All the JSON needs proper indentation (4 spaces instead of 2), for an example check this one:

{
"version": 1,
"license": "CC0-1.0",
"copyright": "Original work by TJohnson.",
"size": {
"x": 32,
"y": 32
},
"states": [
{
"name": "icon"
},
{
"name": "equipped-INNERCLOTHING",
"directions": 4
},
{
"name": "inhand-left",
"directions": 4
},
{
"name": "inhand-right",
"directions": 4
},
{
"name": "rolled-equipped-INNERCLOTHING",
"directions": 4
}
]
}

changes done!

@Possumongus Possumongus requested a review from Radezolid February 2, 2025 18:14
# - CommonBackpack
# - CommonSatchel
# - CommonDuffel
# Start DeltaV changes
Copy link
Contributor

Choose a reason for hiding this comment

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

This should start before the line 181

@@ -24,6 +24,22 @@
id: HoPBoatswain
equipment:
jumpsuit: ClothingUniformJumpsuitBoatswain

Copy link
Contributor

Choose a reason for hiding this comment

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

Remove these four spaces

Copy link
Contributor

Choose a reason for hiding this comment

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

The whole file is over indented by 1 space.

Comment on lines +5 to +7
"size": {
"x": 32,
"y": 32
Copy link
Contributor

Choose a reason for hiding this comment

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

This is over indented by a total of 5 spaces, size should be at the same height that copyright, remove the space I pointed out at the file comment and then fix the indentation.

Copy link
Contributor

Choose a reason for hiding this comment

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

This file is overindented by one space.

Copy link
Contributor

Choose a reason for hiding this comment

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

JSON over indented by one space.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Sprite Changes any png or json in an rsi Changes: YML Changes any yml files S: Needs Review size/M 64-255 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants