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

player display: adds support for hidden traits #27432

Merged
merged 3 commits into from
Jan 5, 2019

Conversation

mlangsdorf
Copy link
Contributor

@mlangsdorf mlangsdorf commented Jan 4, 2019

Summary

SUMMARY: Interface "player display: add support for hidden traits"

Purpose of change

Part of addressing #27397
Add the ability for traits to not be displayed in the '@' player display.

Describe the solution

Add a new player_display field to a mutation_branch, which defaults to true, and load it from JSON.

Change get_mutations() to take an optional boolean, include_hidden, which defaults to true. If it is false, traits with player_display set to false are not returned by get_mutations(). Pass false to the get_mutations() call in player_display.cpp.

Set "player_display": false the NPC only traits for the Tacoma Commune and the survivor background story traits.

Describe alternatives you've considered

I could have added a specialized get_display_mutations() call but I don't think that's necessary at this time.

Change the syntax highlighting to C++, since that produces more
attractive results with the inline comments.
Add support for hidden traits that are not displayed in the '@' menu.
Hide the NPC only traits and the survivor story traits.
@mlangsdorf mlangsdorf added Info / User Interface Game - player communication, menus, etc. [JSON] Changes (can be) made in JSON Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies [C++] Changes (can be) made in C++. Previously named `Code` labels Jan 4, 2019
@mlangsdorf
Copy link
Contributor Author

jenkins rebuild

@ZhilkinSerg ZhilkinSerg merged commit 7c175a6 into CleverRaven:master Jan 5, 2019
@mlangsdorf mlangsdorf deleted the hidden_traits branch January 5, 2019 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc. [JSON] Changes (can be) made in JSON Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants