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

un-variant eye color mutation #70015

Closed
wants to merge 2 commits into from
Closed

Conversation

KeremBabaG
Copy link
Contributor

@KeremBabaG KeremBabaG commented Dec 5, 2023

Summary

Bugfixes "fixes eye color mutation"

Purpose of change

Closes #69944. Eye color mutation for some reason was a single entry with multiple variants, which prevented people from changing to a specific eye color via mirrors or debug.

Describe the solution

Change eye colors to be 6 different mutations with different variants instead of one mutation with 6 variants, adjust npcs accordingly.

Describe alternatives you've considered

don't do this

Testing

the game gave a warning about eye_color being an invalid trait, but otherwise nothing else for now.

Additional context

Ekran görüntüsü 2023-12-06 012628

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies <Bugfix> This is a fix for a bug (or closes open issue) astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Dec 5, 2023
@KeremBabaG KeremBabaG marked this pull request as draft December 5, 2023 22:36
@github-actions github-actions bot added NPC / Factions NPCs, AI, Speech, Factions, Ownership and removed json-styled JSON lint passed, label assigned by github actions labels Dec 5, 2023
@Vaskritaya
Copy link
Contributor

I'm not convinced this is the best way of doing this. The hair variants have a similar problem in debug and with the haircut kit, and it would be terrible for maintenance and json cleanliness to decompose them like this too. Ideally, we'd have more robust handling of mutation variants in these kinds of uis.

It's not a huge issue for the handful of eye mutations though. Either way, just my two cents.

@worm-girl
Copy link
Contributor

worm-girl commented Dec 5, 2023

I'm not convinced this is the best way of doing this. The hair variants have a similar problem in debug and with the haircut kit, and it would be terrible for maintenance and json cleanliness to decompose them like this too. Ideally, we'd have more robust handling of mutation variants in these kinds of uis.

It's not a huge issue for the handful of eye mutations though. Either way, just my two cents.

I do think the whole thing could be done better. Maybe what we need is a better and more consistent way in code to swap between variants. This would also let us add hair dye.

I think this should be closed and #70017 should be the model we work from.

@KeremBabaG KeremBabaG closed this Dec 6, 2023
@KeremBabaG
Copy link
Contributor Author

kk closed

@KeremBabaG KeremBabaG deleted the mirrors branch December 6, 2023 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies NPC / Factions NPCs, AI, Speech, Factions, Ownership
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Standing mirror changes eye color randomly
3 participants