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

New Synth Customisation Options #500

Merged
merged 25 commits into from
Jul 8, 2024
Merged

Conversation

Constellado
Copy link
Contributor

@Constellado Constellado commented Jun 10, 2024

Adds options for synth heads!

  1. There is a new organ that goes on the head of the synth, and acts as a kind of "cover" that goes over their head, augmenting how they look!
  2. I changed what the default synth head looks like, it is now like a skeleton. This head has eyes! Which you can recolour.
  3. Synths also now have a pickable gender!

Without a head cover:
image
image
image

Sprites are from me apart from the IPC sprite, which is from "icons\mob\clothing\head\costume.dmi".
Helm default:
image
image
image

Helm white:
image
image
image

IPC (was from somewhere else in the code base)
(notice! It hides hats due to the huge size!)
image
image
image

Also this option!!
image
image

@Constellado
Copy link
Contributor Author

I got it working! It just cannot render under hats. If I set layers = EXTERNAL_ADJACENT it just dissapears. It basically has to be on the top layer??? Idk whats going on there.

@Constellado Constellado changed the title New Synth Head Options [DRAFT] [NOT WORKING] New Synth Customisation options [DRAFT] Jun 11, 2024
@Constellado Constellado changed the title New Synth Customisation options [DRAFT] New Synth Customisation Options [DRAFT] Jun 11, 2024
@Constellado Constellado changed the title New Synth Customisation Options [DRAFT] New Synth Customisation Options\ Jun 11, 2024
@Constellado Constellado changed the title New Synth Customisation Options\ New Synth Customisation Options Jun 11, 2024
@Constellado Constellado marked this pull request as ready for review June 12, 2024 06:25
Comment on lines +516 to +518
/datum/sprite_accessory/synth_head_cover/none // for those that don't want a cover.
name = "None"
icon_state = null
Copy link
Owner

Choose a reason for hiding this comment

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

I don't think you need an bespoke type for this, I think instead all you need to do is when you make the global list
(init_sprite_accessory_subtypes), you add add_blank = TRUE to the call, and that adds an empty sprite accessory for you

Copy link
Contributor Author

Choose a reason for hiding this comment

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

doing that seems to introduce a bug where the preferences menu shows a head cover applied to the option that is supposed to be blank.
Not sure what that is about so I will hold back on this for now.

@MrMelbert MrMelbert merged commit 697ef4c into MrMelbert:master Jul 8, 2024
21 checks passed
@Constellado Constellado deleted the SynthHeads branch July 30, 2024 11:01
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