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

Change: Default settings improved for new players #8463

Merged
merged 1 commit into from Jan 2, 2021

Conversation

@2TallTyler
Copy link
Member

@2TallTyler 2TallTyler commented Dec 29, 2020

Motivation / Problem

In #8393, nine community members discussed the best default settings for new players who have not yet waded into the settings menu we have all tweaked to our liking. We agreed on changing six settings which are currently unfriendly to new players.

Description

Each of these received a unanimous agreement of at least 6 votes, and most received 8-10.

  • Savegame date format (ISO)
  • Group expenses in company finance window (on)
  • Inflation (off)
  • Quick creation of vehicle orders (on)
  • New orders are non-stop by default (on)

Additionally, one issue received 10 upvotes and one downvote.

  • Show path reservations for tracks (on)

Limitations

Two settings received four upvotes and no downvotes, but I don't feel like this is enough to justify changing the default settings.

  • Interface > Construction > Link landscape toolbar to rail/road/water/airport toolbars
  • Environment > Industries > Allow multiple similar industries per town

There was also discussion about breakdowns and train/road vehicle collisions at level crossings, but the consensus seemed to be that these issues should be fixed separately rather than ignored by changing default settings.

Copy link
Member

@TrueBrain TrueBrain left a comment

I think it is fair if that many people responded positive to change the default, to accept it.

I would like another developer to merge it if he agrees with the same :)

@LordAro
LordAro approved these changes Jan 2, 2021
Copy link
Member

@LordAro LordAro left a comment

Sure, why not. YOLO an all.

@LordAro LordAro merged commit fe51051 into OpenTTD:master Jan 2, 2021
7 checks passed
7 checks passed
Emscripten
Details
Commit checker
Details
Linux (clang, clang++)
Details
Linux (gcc, g++)
Details
Mac OS (x64, x86_64) Mac OS (x64, x86_64)
Details
Windows (x86) Windows (x86)
Details
Windows (x64) Windows (x64)
Details
@2TallTyler 2TallTyler deleted the 2TallTyler:better_default_settings branch Jan 2, 2021
@2TallTyler 2TallTyler restored the 2TallTyler:better_default_settings branch Feb 24, 2021
@2TallTyler 2TallTyler deleted the 2TallTyler:better_default_settings branch Feb 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants