Skip to content

Add soil tooltips and improve wrong soil chat messages. - #263

Merged
C0bra5 merged 10 commits into
masterfrom
soil-tooltips
Sep 20, 2026
Merged

C0bra5 merged 10 commits into
masterfrom
soil-tooltips

Conversation

@C0bra5

@C0bra5 C0bra5 commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

This PR improves the current messaging around soil requirements. It does so by doing 2 things:

  1. All seeds now show a generalized list of soils they can be planted in.
  2. When planting a seed on the wrong soil, the game now tells the player what soil to use.
    • Unanalyzed seeds get a seperate message that doesn't indicate the soil type, encouraging players to analyze the seed to figure out its soil type.
    • As per usual, alternate seeds (a.k.a. things you can plant to still get a 1/1/1 seed like saplings, and regular seeds) are counted as default analyzed seeds, and therefore use the same message as an analyzed seed when using the wrong soil.

Note

This definitively falls under new features, but I'd be willing to push it for the 2.9 release if someone is willing to review it, since it does do a bit of cleanup in the surrounding code.

Screenshots

Analyzed seed response:

image image

Not analyzed seed:

image image

Video Showcase

java_hmbYg5wWPC.mp4

Checklist

  • I have tested this PR in DevEnv
  • I have tested this PR in Fullpack
  • This PR is in compliance with the GTNH AI Policy
    • No AI used
  • ❌ This PR requires another PR in order to merge

@C0bra5 C0bra5 added Enhancement Improve an existing mechanic. Please explain the change with a before/after comparison. New Feature Add something new. Please explain in detail how it works. labels Sep 14, 2026

@LazyFlesh LazyFlesh left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

other than the one comment, lgtm

Comment thread src/main/java/com/gtnewhorizon/cropsnh/api/ISoilList.java Outdated
@Dream-Master

Copy link
Copy Markdown
Member

@C0bra5

@C0bra5
C0bra5 merged commit 41a575d into master Sep 20, 2026
1 check passed
@C0bra5
C0bra5 deleted the soil-tooltips branch September 20, 2026 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement Improve an existing mechanic. Please explain the change with a before/after comparison. New Feature Add something new. Please explain in detail how it works.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants