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

Paramedic hardhat rework #20340

Closed

Conversation

FabianK3
Copy link
Contributor

@FabianK3 FabianK3 commented Jan 8, 2025

Summary

This PR reworks the paramedic helmet. The intent is to modernize the rather outdated and rather unfavored current version of the paramedic helmet. Besides the icon update, it also has been renamed to hardhat and it's description has been updated to emphasize the fact that it's a hardhat, not an armored helmet. Part of the icon rework is the addition of emissives.

The new design is heavily inspired by the generic security helmet as i found it to be the most generic helmet, with the smallest area to fit the hardhat style. It is also inspired by todays modern hardhats seen in emergency services. All icons have been made by myself and are based of existing designs in the game.

Additionally this PR contains a typo-fix for the hydraulic rescue tool.

Preview

Item
image

Front
image

Side right
image

Side left
image

Back
image

Front emissive
image

Side right emissive
image

Side left emissive
image

Back emissive
image

Front flashlight
image

Side right flashlight
image

@BotBOREALIS BotBOREALIS added the Sprites Adds new or changes existing sprites. label Jan 8, 2025
@FabianK3
Copy link
Contributor Author

FabianK3 commented Jan 8, 2025

!review

Copy link
Contributor

@alsoandanswer alsoandanswer left a comment

Choose a reason for hiding this comment

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

I'm gonna have to use my my jurisdiction has head spriter here - this sprite is a downgrade from the original.
Feel free to change the description, names and add emissives, but I will not permit a change on the original sprite.

@alsoandanswer alsoandanswer added the Changes Required The PR requires changes before it can be approved and/or merged. label Jan 8, 2025
@FabianK3
Copy link
Contributor Author

FabianK3 commented Jan 8, 2025

Could you elaborate your reasons for denying modifications on that sprite or why it is a downgrade in your opinion?
Modern hardhats become more and more lighter, the current version looks like an outdated combat helmet which makes not much sense for it's purpose.

Respectfully, i haven't met single a paramedic yet that remotely expressed liking to the original sprite.

@alsoandanswer
Copy link
Contributor

alsoandanswer commented Jan 9, 2025

Could you elaborate your reasons for denying modifications on that sprite or why it is a downgrade in your opinion?

dreammaker_PKCBYV0vXP

You've very obviously sprited over the security helmet sprite, which is very old, mind you - we're trying to move away from that and update our sprites from old 2010-era /tg/ sprites.

Not only that, I've sprited the paramedic hardhat specifically to not look like the security helmet so that it wouldn't be confused that it would have any combat utility whatsoever.

You've even wrote that in the extended description.

Its purpose is to protect the wearer from falling things and bumping their head, it's not rated for combat action.

So, not only did your new sprite fail at conveying that it is a hardhat, you also needed to extend the description to explicitly state that it is, in fact, not a combat helmet.

@FabianK3
Copy link
Contributor Author

FabianK3 commented Jan 9, 2025

Please see the PR summary.

The new design is heavily inspired by the generic security helmet as i found it to be the most generic helmet, with the smallest area to fit the hardhat style. It is also inspired by todays modern hardhats seen in emergency services.

Yes, i used the generic helmet.
Why? It feels 10x lighter then the current sprite. Why? The generic helmet feels much smaller in terms of its actual sprite size, resulting in less association with armor, ironically. The current helmet is too big to not feel heavy (heavy as in armored).

Example:
The new sprite feels lighter, i had this in mind:

image

The old sprite, especially with the ear protection curvatures, looks very very much like an actual swat helmet:

image

So, not only did your new sprite fail at conveying that it is a hardhat, you also needed to extend the description to explicitly state that it is, in fact, not a combat helmet.

I have added the description not because i think the new version feels like combat gear, but because i still had the old sprite in my head, which feels like combat gear.

Not only that, I've sprited the paramedic hardhat specifically to not look like the security helmet so that it wouldn't be confused that it would have any combat utility whatsoever.

Ironically, i think the generic helm even looks less combat related than the current paramedic helmet sprite.

@FabianK3 FabianK3 requested a review from alsoandanswer January 9, 2025 01:07
@FabianK3
Copy link
Contributor Author

FabianK3 commented Jan 9, 2025

To confirm my point: Shortly after my PR got posted on discord, somebody thought i changed the helmet to an hardhat - There is the proof that people thought the previous sprite was indeed an armored helmet and the newer version looks lighter.

I've also collected some feedback before i opened the PR.

@alsoandanswer
Copy link
Contributor

Look, I'm not going to argue semantics here - we've learnt from the Medical Chemical Names discussion that realism is incompatible with gameplay, so I'm not going to entertain the real-life comparisons here as well.

For the feedback, I'm going to give benefit of the doubt and take it at face value : Paramedics don't like the helmet. True, but they didn't like the old one either. Namely because other options like caps and showing off your hair are simply more popular.
Also, the new sprite is designed to work alongside the job uniforms in mind, with the pallette to match as well. If the helmet doesn't work with someone's custom outfit, that's to be expected. I can't possibly keep up with the dozens of loadout item ports from other servers with dubious artstyles, and even then I'm going to resprite them anyway down the line. If the helmet is truly pivotal to their character, I'd recommend them to apply for a custom item.

As for the "Is this combat or hardhat", it's purely subjective. But for me, it's based on existing conventions. It looks like the security helmet. Therefore, it looks like it's used for combat. The existing sprite looks big and bulkly. The hardhats are also big and bulky. Therefore, it is a hardhat.

And now, as feedback is important to you, is my personal feedback on the sprite.

Fundementally, as a sprite, it's very amateurish - please read the sprite guide on the forums to understand my thought process.

And on a more important note - this is just a straight-up unintentional reversion to the pre-resprite version, if you look through the file history. I'm not going to allow what is basically a cyclical resprite back to the 2010s. I'm not going to allow this server's artstyle to be stuck in the past any longer.

@FluffyGhoster
Copy link
Contributor

I agree with Wezzy, this would need to either not touch the sprite, or provide a better one than what we currently have

@FabianK3
Copy link
Contributor Author

FabianK3 commented Jan 9, 2025

Look, I'm not going to argue semantics here - we've learnt from the Medical Chemical Names discussion that realism is incompatible with gameplay, so I'm not going to entertain the real-life comparisons here as well.

I wasn't trying to pull semantics on anyone. I'm also unaware of the discussion you are referring to.

For the feedback, I'm going to give benefit of the doubt and take it at face value : Paramedics don't like the helmet. True, but they didn't like the old one either. Namely because other options like caps and showing off your hair are simply more popular.
Also, the new sprite is designed to work alongside the job uniforms in mind, with the pallette to match as well. If the helmet doesn't work with someone's custom outfit, that's to be expected. I can't possibly keep up with the dozens of loadout item ports from other servers with dubious artstyles, and even then I'm going to resprite them anyway down the line. If the helmet is truly pivotal to their character, I'd recommend them to apply for a custom item.

I certainly do not expect you to come up with sprites that suits every type of loadout, artstyle or others.

As for the "Is this combat or hardhat", it's purely subjective. But for me, it's based on existing conventions. It looks like the security helmet. Therefore, it looks like it's used for combat. The existing sprite looks big and bulkly. The hardhats are also big and bulky. Therefore, it is a hardhat.

I understand your argument, but must mention that this view doesn't seem to be what others players think about the helmet. The issue was the reason for me to try to present an alternative.

Fundementally, as a sprite, it's very amateurish - please read the sprite guide on the forums to understand my thought process.

It is indeed amateurish because this is the first time i have edited a sprite and i think for a basic reskin i don't think it's to shabby. I sadly am not sure what i could've done differently, even with the guide on hand. Perhaps that is my issue without experience in the matter.

And on a more important note - this is just a straight-up unintentional reversion to the pre-resprite version, if you look through the file history. I'm not going to allow what is basically a cyclical resprite back to the 2010s. I'm not going to allow this server's artstyle to be stuck in the past any longer.

I wasn't aware of such an historical issue with art. This is my first sprite contribution, or attempt rather.


After further consideration i understand that you see no value in using the generic helmet as an alternative. I might try something else in the future if i find the motivation for it after this discussion.

I'm gonna have to use my my jurisdiction has head spriter here - this sprite is a downgrade from the original.

It would be very valuable for any [newer] contributor when you pull the jurisdiction card to elaborate on your thoughts instead of calling it a simple downgrade. It helps people to understand the direction you like to see in relation to the work presented. We could've saved us the semantics discussion too.

Thank you for your time.

@FabianK3 FabianK3 closed this Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes Required The PR requires changes before it can be approved and/or merged. Review Required Sprites Adds new or changes existing sprites.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants