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

Doc: revised README to be more helpful to casual GitHub visitors #7807

Merged
merged 1 commit into from Nov 13, 2019

Conversation

andythenorth
Copy link
Contributor

Think this is ready for nitpicking.

Note for reviewers: there's a lot of prior discussion about this on irc, anyone approaching it cold might find they're lacking the full tedious backstory

  • the goal is to make the readme more approachable and useful to GitHub casual visitors, without losing the ability to read it locally in the distributed bundle
  • this may also end being visible in-game via Add: Help and manuals window #7786 but that is a separate concern
  • we're comfortable linking to remote pages that require an internet connection

For minor typos, it would be helpful to just fix them and force push. Putting typo change requests on the GH PR review is pointless triple-handling for fixing minor typos. Thx!

README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
@andythenorth andythenorth force-pushed the revise-readme-october-2019 branch 10 times, most recently from 3ef1769 to 4da1ea0 Compare October 30, 2019 21:01
@andythenorth andythenorth force-pushed the revise-readme-october-2019 branch 3 times, most recently from 24685e0 to f904bd0 Compare November 9, 2019 15:29
Copy link
Member

@michicc michicc left a comment

Choose a reason for hiding this comment

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

👍

README.md Outdated Show resolved Hide resolved
michicc
michicc previously approved these changes Nov 9, 2019
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved

#### 4.1.1) Free graphics and sound files
### 1.4.1) Free graphics and sound files

The free data files, split into OpenGFX for graphics, OpenSFX for sounds and
Copy link
Member

Choose a reason for hiding this comment

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

Section might want a rework - very repetitive "Open*FX" and bare links

Copy link
Contributor Author

Choose a reason for hiding this comment

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

+1 but no ideas for solutions...

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved

Game scripts can provide additional challenges or changes to the standard OpenTTD gameplay, for example setting transport goals, or changing town growth behaviour.

Game scripts can be added via the 'Check Online Content' button in the main menu.
Copy link
Member

Choose a reason for hiding this comment

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

Very very repetitive "can be added via the ..." Put into parent section.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is very repetitive, but the information under each sub-heading has to stand alone as instructions. Can't rely on referring up, or assuming the reader has read or retained earlier information.

I tried to think of a better way, but this was it so far.

README.md Outdated Show resolved Hide resolved
@michicc michicc merged commit cfc4568 into OpenTTD:master Nov 13, 2019
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.

None yet

5 participants