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

Update main menu tips for BN #1454

Merged
merged 5 commits into from
Apr 21, 2022
Merged

Conversation

chaosvolt
Copy link
Member

@chaosvolt chaosvolt commented Apr 11, 2022

Summary

SUMMARY: Interface "Update main menu tips to account for changes in BN, new mechanics, etc"

Purpose of change

A while back it was suggested to update the tips available in BN, as some of them concern mechanics specific to DDA, and of course some mechanics specific to BN lack a mention. Plus can add a few for advice concerning a few mechanics common to both that don't get much hinting.

Describe the solution

  1. Removed four tipd concerning frozen food.
  2. Added a tip pertaining to the electric grid system, and changed the oven tip to allude to being able to restore power.
  3. Added a tip concerning the different behavior of mutated meat.
  4. Changed tip concerning vehicles to not refer to tanks, since those have never even been vanilla.
  5. Added a tip concerning helicopters, and how they can be repaired.
  6. Added a tip alluding to the endgame lab.
  7. Added a tip concerning the usefulness of firewood sources.
  8. Added a tip pertaining to just what you can do with deathmobiles, per suggestion of @EkarusRyndren. Alluding to their particular flavor of over-the-top deathmobile is a bonus.

Describe alternatives you've considered

One idea that came up is it'd be nice if we had a key that'd let the player load another tip at random or some way to view all tips, but no idea how to implement that.

Testing

  1. Checked affected file for syntax and lint errors.
  2. Loaded up compiled test build several times to see what the messages look like, to ensure they aren't too long.

Additional context

@@ -162,7 +162,17 @@
{
"type": "snippet",
"category": "tip",
"text": [ "Why walk when you can use a car? Or a tank?" ]
"text": [ "Mutated wildlife is still edible, but not without risks to your own DNA." ]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cata mutations aren't necessarily DNA. Maybe hint on "you are what you eat"?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could work, yeah.

@Coolthulhu Coolthulhu self-assigned this Apr 19, 2022
@@ -42,7 +42,7 @@
{
"type": "snippet",
"category": "tip",
"text": [ "Frozen food? Put it by a fire or heat it up in a pan." ]
"text": [ "The power may be out, but by constructing solar panels you can make your own." ]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sounds weird. "The power"?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should I change to "The electricity may be out," or "Power may be out," for this?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something like this. Also the "but you can make your own" sounds weird in this context. It makes it sound kinda tangible.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True, hmm.

@@ -42,7 +42,7 @@
{
"type": "snippet",
"category": "tip",
"text": [ "Frozen food? Put it by a fire or heat it up in a pan." ]
"text": [ "Power may be out, but by constructing solar panels you can generate your own elecricity." ]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should probably get an extra comma in there.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After but by constructing solar panels I assume?

@Coolthulhu Coolthulhu merged commit 205c22b into cataclysmbnteam:upload Apr 21, 2022
@chaosvolt chaosvolt deleted the tip-update branch April 21, 2022 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants