-
Notifications
You must be signed in to change notification settings - Fork 722
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
Stardew Valley: Options page documentation improvements #3155
Stardew Valley: Options page documentation improvements #3155
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Looks fine to me. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, mainly shortening things up
Some of these option descriptions have questions marks, which makes them read strangely. Some of them much more than others and there are some periods missing from the end of some sentences, but those are out of scope for this change. The actual changes here seem good to me |
What is this fixing or adding?
1: Fixed a typo (leaning -> learning)
2: Shortened the goal documentation so that the tooltip is more readable (hopefully)
How was this tested?
Documentation