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

Adds weapon icons to the game #21995

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

Conversation

AMyriad
Copy link
Contributor

@AMyriad AMyriad commented May 2, 2024

Document the changes in your pull request

This is part 1 of the project where I organize, reorganize, and clean up all our icon files.

Atomized for the sake of our spritetainers' sanity (I'm sorry), this PR focuses on organization of melee weapons.
Know that I've double checked everything to make sure I didn't break anything, but I'll test later to triple check ofc.

Why is this good for the game?

Redundant/unused icons, "snowflake" icon files with a single icon, and messy icon files are not good.
Sorting these things out helps icon file readability for contributors and will over time improve game performance, since we'll be grouping more relevant things in the same files, lightening the load for when the game draws all the icons from each file.

Testing

image

Spriting

Before
image

After
image
( more things in the same file -> icons less spread out -> better )

Changelog

🆑
experimental: Reorganized our weapon icons. Ping me or make a bug report if you see any weapon missing their sprites!!
/:cl:

@AMyriad AMyriad requested a review from a team as a code owner May 2, 2024 20:04
@AMyriad AMyriad added Sprites This PR has spriting changes Awaiting - Action - Sprite Review Awaiting sprite review from sprite maintainers labels May 2, 2024
@Chubbygummibear
Copy link
Contributor

i'm rather opposed to deleting any yogstation dmi that has a mirrored version in the main folder even if it's only 1 file in a folder, because the point is that there's a place to add sprites that are unique to yog. like in the case of the yog lavaland artifacts. it should remain so that if we add more artifacts or @LazennG makes more yog unique lavaland loot, there's a place to put it that's reflective of the fact it's an asset unique to us. it's always a weird feeling when you have to be the first one to make the new dm or dmi in the yogstation/ directory because there's no food.dm or vending.dmi or whatever, but someone's gotta be the first, even if it only is for 1 sprite or 1 subtype.

@AMyriad
Copy link
Contributor Author

AMyriad commented May 10, 2024

i'm rather opposed to deleting any yogstation dmi that has a mirrored version in the main folder even if it's only 1 file in a folder, because the point is that there's a place to add sprites that are unique to yog. like in the case of the yog lavaland artifacts. it should remain so that if we add more artifacts or LazennG makes more yog unique lavaland loot, there's a place to put it that's reflective of the fact it's an asset unique to us. it's always a weird feeling when you have to be the first one to make the new dm or dmi in the yogstation/ directory because there's no food.dm or vending.dmi or whatever, but someone's gotta be the first, even if it only is for 1 sprite or 1 subtype.

Alright I understand that, but I don't understand why we have yog-unique stuff in a subfolder at all. There are plenty of things unique to yogs outside of that folder and vice versa, and right now it's filled with a lot of old, outdated, misplaced icons in strange misnamed places (like /xenoarch/guns.dmi for example) that haven't been maintained in ages.

I strongly believe it would be better to find our files in one place, as is - from my understanding - usually the purpose of organizing files. I don't see why separating things purely because they're unique to here is good practice when they could be in the "normal" files like everything else? What would that say about everything outside of the yogstation folder, is that not ours either?

@Mqiib
Copy link
Contributor

Mqiib commented May 23, 2024

image
image

What's the edit to the geiger counter, and why is the hammer renamed?

Copy link
Contributor

@Mqiib Mqiib left a comment

Choose a reason for hiding this comment

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

right i need to hit this button, see above

@AMyriad
Copy link
Contributor Author

AMyriad commented May 26, 2024

What's the edit to the geiger counter, and why is the hammer renamed?

I didn't change anything about that geiger icon but there's a chance I moved it in the file by accident. I checked and didn't see anything different, but just to be safe I copy pasted the icon from master so there shouldn't be any problemo now

For the hammer, I thought calling something "hammer" in hammer.dmi already would be a bit vague, and it's what TG calls it in their files so why not ¯\_(ツ)__/¯

@Mqiib
Copy link
Contributor

Mqiib commented May 26, 2024

icondiff bot is mad tho

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting - Action - Sprite Review Awaiting sprite review from sprite maintainers Sprites This PR has spriting changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants