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

Ports 8 hairstyles from Citadel #19204

Closed
wants to merge 2 commits into from

Conversation

listerla
Copy link
Contributor

From Citadel-Station-13/Citadel-Station-13-RP#6109

To address the elephant in the room; yes, these are reference hairs. I think they should get in anyway because they look good, we don't have enough good hairstyles, and we literally have Vegeta hair titled 'Vegeta' ingame already. I can rename them if you suggest new names, if that is preferred. I just couldn't think of any.

@listerla listerla requested a review from alsoandanswer as a code owner May 20, 2024 11:30
@BotBOREALIS BotBOREALIS added the Sprites Adds new or changes existing sprites. label May 20, 2024
@listerla
Copy link
Contributor Author

!review

@Stev-the-third
Copy link
Contributor

It somewhat feels as if all five of these hairstyles should just be "Blank 1 - 5", since they're so similar. (Or at least the bottom 4, unsure about the top one.)
image

@listerla listerla closed this May 20, 2024
@listerla listerla deleted the signalishair branch May 20, 2024 11:45
@listerla listerla restored the signalishair branch May 31, 2024 20:40
github-merge-queue bot pushed a commit that referenced this pull request Jun 9, 2024
From
Citadel-Station-13/Citadel-Station-13-RP#6109

To address the elephant in the room; yes, these are reference hairs. I
think they should get in anyway because they look good, we don't have
enough good hairstyles, and we literally have Vegeta hair titled
'Vegeta' ingame already. I can rename them if you suggest new names, if
that is preferred. I just couldn't think of any.

Accidentally deleted the branch. Oopsies! Old PR:
#19204
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Review Required Sprites Adds new or changes existing sprites.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants