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

Vulpkanin's Deuteranopia #837

Merged
merged 2 commits into from
Feb 15, 2024
Merged

Conversation

FluffiestFloof
Copy link
Contributor

@FluffiestFloof FluffiestFloof commented Feb 15, 2024

Everyone loves dog vision :trollface:

About the PR

Adds Deuteranopia as a selectable trait.
Adds Deuteranopia to Vulpkanin by default. It can be reverted using the Normal Vision trait similarly to the Harpy UltraVision.
Make Normal Vision and Ultraviolet Vision trait names into locale.

Why / Balance

Why not, Harpy gets a default vision and it make sense for Vulpkanins.
I'm sure a lot of people won't like it, even less when the Metempsychotic machine gets readded and they roll into a Vulpkanin.

Technical details

I don't think I really needed to remove all those funny lines but the UltraVision doesn't need em and since you can't really gain/lose Deuteranopia anyway outside of selecting the trait or being Vulpkanin...

Media

ss (2024-02-15 at 11 41 31)

  • [yes] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

🆑

  • add: Added Deuteranopia trait.
  • tweak: Vulpkanin now start with Deuteranopia by default which can be removed using the Normal Vision trait.

@github-actions github-actions bot added S: Needs Review Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: YML Changes any yml files labels Feb 15, 2024
@AitorLogedo
Copy link
Contributor

the piss vision

@VMSolidus
Copy link
Contributor

Yea, removing those lines is fine. Ultravision originally copied all the Dogvision parts, but within seconds of having it up, I realized, "Half this code is unused, what the hell?", and cleaned it up by stripping out everything unneeded. I would have gone back and cleaned up dogvision too, had I not been concerned about scope creep.

This is fine.

@VMSolidus
Copy link
Contributor

In fact removing all those lines doesn't change the feature functionality in any way. They're totally vestigial. The removed code was in fact ADDING a bug by inherently tying component initialization/deinit to having a player session, thus making it so that taking over a creature with DogVision meant you wouldnt have it since it didnt initialize on your player session.

I am strongly considering adding an implant that just gives you the NormalVision component.

@FluffiestFloof
Copy link
Contributor Author

In fact removing all those lines doesn't change the feature functionality in any way. They're totally vestigial. The removed code was in fact ADDING a bug by inherently tying component initialization/deinit to having a player session, thus making it so that taking over a creature with DogVision meant you wouldnt have it since it didnt initialize on your player session.

I am strongly considering adding an implant that just gives you the NormalVision component.

Nice good to know I pre-emptively fixed something.

@DebugOk DebugOk merged commit a98b4eb into DeltaV-Station:master Feb 15, 2024
12 checks passed
DeltaV-Bot pushed a commit that referenced this pull request Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: YML Changes any yml files S: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants