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

Hair Edits #1209

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

Hair Edits #1209

wants to merge 9 commits into from

Conversation

Doctor-Derp
Copy link

@Doctor-Derp Doctor-Derp commented Feb 15, 2025

About The Pull Request

Tidies up the hair.dmi folder to reflect the alphabetical order enforced in the hair code file.
Adds the following hairstyles to the game, ported from Nova + Iris Station.

Bob Hair (Long)

Bob Hair (Long)

Bob Hair (Medium)

image

Bob Hair (Short)

image

Braid (Medium)

image

Braid (Mermaid)

image

Bun (Elegant)

image

Bun (Gentle)

image

Bun (Holotuber)

image

Bun (Low)

image

Bun (Tassles)

image

Bun (Tri Long)

image

Bun (Tri)

image

Cornrows

image

Cornrows (Long)

image

Over Eye (Messy)

image

Ponytail (Side) 6

image

Ponytail (Simple Alt)

image

Ponytail (Thick)

image

Ponytail (Thin Short)

image

Ponytail (Thin)

image

Ponytail (Zoey)

image

Royal Curls

image

Runner

image

Runner Bun

image

Very Long Hair (Over Eye Alt)

image

Why It's Good For The Game

I noticed there was a distinct lack of good hairstyles, and there was some styles I love only found on Iris. I also noticed a few duplicate hairstyles, and how unorganized the dmi file was. These issues have been rectified.

Changelog

🆑
add: Added 25 new hairstyles.
del: Removed duplicate hairstyles.
qol: icon_states present in hair.dmi are now in alphabetical order.
code: Added a hairstyle preference save migration.
code: Several hairstyle code paths and names have been sorted and simplified.
/:cl:

@Doctor-Derp Doctor-Derp marked this pull request as draft February 15, 2025 08:45
@Doctor-Derp Doctor-Derp marked this pull request as ready for review February 15, 2025 09:07
Copy link
Member

@francinum francinum Mar 7, 2025

Choose a reason for hiding this comment

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

This file isn't included. Tick it in your editor to add it to the DME.

Copy link
Author

Choose a reason for hiding this comment

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

I had assumed I shouldn't touch the dme file, oopsie. I've ticked it and pushed the dme change, does this look right?

@Doctor-Derp Doctor-Derp requested a review from francinum March 12, 2025 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants