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

Security Vest and Helmet Resprite #30109

Conversation

DieselMohawk
Copy link
Contributor

@DieselMohawk DieselMohawk commented Jul 17, 2024

About the PR

Resprites the default Security vest and helmet.

Why / Balance

The old variations were old and were beginning to fall behind the rest of the art style, plus I want to see usage of the vest and helmet as they have been mostly unused by anyone except cadets.

Technical details

Media

New:
image
image
image
image
Old:
image
image
image
image

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

🆑

  • tweak: Security Vest Resprited
  • tweak: Security Helmet Resprited

@github-actions github-actions bot added 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. labels Jul 17, 2024
Copy link
Contributor

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

Resources/Textures/Clothing/Head/Helmets/security.rsi

State Old New Status
equipped-HELMET-vox Modified
equipped-HELMET Modified
icon Modified
inhand-left Modified
inhand-right Modified

Resources/Textures/Clothing/OuterClothing/Armor/security.rsi

State Old New Status
equipped-OUTERCLOTHING-vox Modified
equipped-OUTERCLOTHING Modified
icon Modified
inhand-left Modified
inhand-right Modified

@ps3moira
Copy link
Contributor

I don't like the flatness of the previous resprite for the helmet, but these changes don't look right, mainly the vest itself.

@Brandon-Huu
Copy link
Contributor

Let's make them look less evil not more

@DieselMohawk
Copy link
Contributor Author

I don't like the flatness of the previous resprite for the helmet, but these changes don't look right, mainly the vest itself.

What about the vest is not desirable?

@DieselMohawk
Copy link
Contributor Author

DieselMohawk commented Jul 17, 2024

Let's make them look less evil not more

Would you mind explaining what makes you say that?

@KittenColony
Copy link
Contributor

The vest looks like shit, can't exactly pinpoint what, it's better then the original. The main issue is when you paint shit, its still shit

the helmet looks better though

@KittenColony
Copy link
Contributor

Let's make them look less evil not more

Would you mind explaining what makes you say that?

tfw shitsec

@DieselMohawk
Copy link
Contributor Author

I'd like feedback on what does or doesn't work, it's fine to not like it, just tell me why in a way that I can work on.

@ps3moira
Copy link
Contributor

I'd like feedback on what does or doesn't work, it's fine to not like it, just tell me why in a way that I can work on.

Extremely stiff, I understand you were trying to do velcro straps on the sides as well but it just doesn't visually work.

@Blackern5000
Copy link
Contributor

The gradient on the vest is ugly.
The vest is too straight on the sides.
The odd "texture" on the lower half feels wrong.
The outline on the bottom is too thick.

@Golinth
Copy link
Contributor

Golinth commented Jul 17, 2024

I feel like there isn’t enough depth(?) to the inventory sprite. Like with the shading and the shoulders, it’s very clear the old one is a representation of a physical object, while the new one has almost none of that, and looks closer to a painted cardboard cutout

@UbaserB
Copy link
Member

UbaserB commented Jul 17, 2024

It’s too dark for one, and the form feels off

@DieselMohawk
Copy link
Contributor Author

Alrighty, this should be enough feedback. I'll keep the helmet design and completely rework the vest.
It will remain a relatively dark color to keep with the slim vest that most officers use, this is so that you can use the helmet with either vest and look good.

@IProduceWidgets
Copy link
Contributor

I think the glass visor looks a little weird. I think I'd rather it get removed from the icon than added to the clothing sprites.

@DieselMohawk DieselMohawk deleted the Sec-Vest-And-Helmet-Resprite branch July 23, 2024 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants