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

don't consider good memory and forgetful neutral #78866

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

casswedson
Copy link
Contributor

Summary

None

Purpose of change

because they are listed as neutral like hair color etc. since they have an actual gameplay effect I moved them to the good and bad columns respectively

Describe the solution

Describe alternatives you've considered

Testing

look at the trait selection at character creation

Additional context

because they are listed as neutral like hair color etc. since they have
an actual gameplay effect I moved them to the good and bad columns
respectively
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Dec 31, 2024
@osuphobia
Copy link
Contributor

osuphobia commented Dec 31, 2024

they are listed as neutral like hair color etc

| `vanity` | false | This trait is purely cosmetic, and can be changed at any time. This is for things like skin color, eye color, hair color, etc. |

Neutral does not mean no effect, it's for vanity.
I think the real issues are

  • They have "mixed_effect": true but not shown as purple at character creation.

  • Some hair styles lack "vanity": true.

@harakka
Copy link
Member

harakka commented Dec 31, 2024

Good memory is 0 points on purpose, it has positive and negative effects.

@Fris0uman
Copy link
Contributor

The cost should stay 0 but it'd be nice to separate them from the purely cosmetic traits, maybe adding an extra column for mixed trait? So it would go Positives | Negatives | Mixed | Cosmetics

@RenechCDDA
Copy link
Member

RenechCDDA commented Dec 31, 2024

There are other traits in that column which have non-cosmetic effects (very short and very tall).

Also per my reasoning in #76076 I consider it wrong to highlight good memory as a positive trait and bad memory as a negative one, since their gameplay effects do NOT follow that.

If anything you need to set good memory to be a negative trait, because its effects are wholly negative.

@Maleclypse
Copy link
Member

There are other traits in that column which have non-cosmetic effects (very short and very tall).

Also per my reasoning in #76076 I consider it wrong to highlight good memory as a positive trait and bad memory as a negative one, since their gameplay effects do NOT follow that.

If anything you need to set good memory to be a negative trait, because its effects are wholly negative.

What if we made Good Memory increase skill rust? Since it specifically says "You are able to recall memories more easily." Meaning it's dropped from your attentive mind to long term storage which is where things that are rusting are.

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 [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants