Skip to content

traits rework - #4725

Open
Kandiyaki wants to merge 11 commits into
impstation:masterfrom
Kandiyaki:StarfallTraitMenu
Open

Kandiyaki wants to merge 11 commits into
impstation:masterfrom
Kandiyaki:StarfallTraitMenu

Conversation

@Kandiyaki

Copy link
Copy Markdown

About the PR

  • ported the trait menu reskin from Aurora Song - it now fills the full screen and has a search bar. original PR available at https://codeberg.org/Aurora-Song/Aurora-Song-14/pulls/1522
  • got rid of our traits' "subcategory" system, now each trait has a list of other traits that it disallows. redid currently existing trait YMLs to reflect this.
  • added a system for traits to require other traits to be taken. CURRENTLY UNUSED. could be used to, for example, add a synthetic-exclusive trait that allows access to binary comms
  • added an "allowed species" datafield to traits. CURRENTLY UNUSED. could be used to, for example, add a gray-exclusive trait that turns their blood to soda
  • removed the point cost from the weight traits.

Why / Balance

  • the current trait menu is horrid, with 90% of the screen real-estate taken up by absolutely nothing.
  • this adds all the framework one would need to create more complex traits.
  • the weight traits have built-in upsides and downsides, so a point cost feels unneeded.

Technical details

ough there's a lot

  • cherrypicked the AS trait menu reskin and changed some upstream files so the menu uses that reskin
  • 3 new datafields in TraitPrototype:
    • AllowedSpecies (a list of species that can take the trait)
    • Disallows (a list of traits that cannot be taken if you take this trait)
    • Requires (a list of traits that, before taking this trait, you must take at least one of)

Media

2026-11-09.14-52-39.1.mp4

Requirements

Licensing

  • I give permission for any changes to the repository made in this PR to be relicensed under MIT.

Changelog

🆑

  • tweak: ported Aurora Song's traits menu reskin.
  • tweak: the Slight and Hefty traits no longer cost trait points.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant