Skip to content

Quartermaster job and ID icon change #39259

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jul 29, 2025
Merged

Quartermaster job and ID icon change #39259

merged 2 commits into from
Jul 29, 2025

Conversation

K-Dynamic
Copy link
Contributor

@K-Dynamic K-Dynamic commented Jul 28, 2025 •
edited
Loading

About the PR

  • Changes Quartermaster job and ID icons to recoloured Cargo Technician icons.

Why / Balance

Unlike most heads, QM's icon didn't match with any job icon in their department. This is a holdover from SS13 when QM wasn't considered a command member. It's also hard to say what the QM's icon is supposed to be.

Note I'm not against anyone respriting head icons to be different from their department staff in a future PR (see Captain and HoP for example), I'm just making it 'consistent.'

Technical details

N/A

Media

image Note: See top-left for new QM job and ID icons.

Requirements

Breaking changes

N/A

Changelog

🆑

  • tweak: Quartermaster job and ID icons are now based on Cargo Technicians.

@PJBot PJBot added S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. Changes: Sprites Changes: Might require knowledge of spriting or visual design. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. labels Jul 28, 2025
@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines. label Jul 28, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jul 28, 2025 •
edited
Loading

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

Resources/Textures/Interface/Misc/job_icons.rsi

State Old New Status
QuarterMaster Modified

Resources/Textures/Objects/Misc/id_cards.rsi

State Old New Status
idquartermaster Modified

Edit: diff updated after 31b3542

@Entvari
Copy link
Contributor

Entvari commented Jul 28, 2025

Makes alot of sense, good find.

southbridge-fur
southbridge-fur approved these changes Jul 28, 2025
View reviewed changes
Copy link
Contributor

@southbridge-fur southbridge-fur left a comment

Choose a reason for hiding this comment

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

No code changes but I'm glad it has been changed since I know when I started I often got the QM's icon confused with the HoP's icon since they're both little people icons. This is a welcome change in my opinion.

This PR will still need an art team approval though.

@PJBot PJBot added S: Approved Status: Reviewed and approved by at least one maintainer; a PR may require another approval. and removed S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. labels Jul 28, 2025
@BigfootBravo
Copy link
Contributor

Great change. Always been a bit annoying/difficult differentiating between HoP and QM as my monochromatic characters.

@Killerqu00 Killerqu00 added T: New Feature Type: New feature or content, or extending existing content A: Art Area: Art with no implications for other areas. P3: Standard Priority: Default priority for repository items. and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Jul 28, 2025
@PJBot PJBot removed the size/XS Denotes a PR that changes 0-9 lines. label Jul 29, 2025
@K-Dynamic
Copy link
Contributor Author

Forgot to add my own attribution to the copyright field, just pushed it

@Fildrance Fildrance merged commit 60cf548 into space-wizards:master Jul 29, 2025
11 checks passed
@EthanQix
Copy link

Note I'm not against anyone respriting head icons to be different from their department staff in a future PR (see Captain and HoP for example), I'm just making it 'consistent.'

PR to make the HoP icon a blue broom when

@K-Dynamic
Copy link
Contributor Author

PR to make the HoP icon a blue broom when

Three words: Blue clown icon

@princesspengy princesspengy mentioned this pull request Aug 17, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@southbridge-fur southbridge-fur southbridge-fur approved these changes

@TheShuEd TheShuEd TheShuEd approved these changes

Assignees

No one assigned

Labels

A: Art Area: Art with no implications for other areas. 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. P3: Standard Priority: Default priority for repository items. S: Approved Status: Reviewed and approved by at least one maintainer; a PR may require another approval. T: New Feature Type: New feature or content, or extending existing content

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@K-Dynamic @Entvari @BigfootBravo @EthanQix @southbridge-fur @TheShuEd @Fildrance @PJBot @Killerqu00