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

Installer does not say that Windows 8 and 10 are supported. #8102

Closed
James103 opened this issue May 1, 2020 · 2 comments
Closed

Installer does not say that Windows 8 and 10 are supported. #8102

James103 opened this issue May 1, 2020 · 2 comments

Comments

@James103
Copy link
Contributor

@James103 James103 commented May 1, 2020

Version of OpenTTD

1.10.1 on Win 10

Expected result

Installer says that Windows 8 and 10 are supported.
"OpenTTD 1.10.1 for Windows XP, Vista, 7, 8, and 10"

Actual result

Installer does not say that Windows 8 and 10 are supported.
"OpenTTD 1.10.1 for Windows XP, Vista, and 7"

Steps to reproduce

  1. Uninstall OpenTTD.
  2. Reinstall OpenTTD from the installer.
  3. Look at the OS version component of the "OpenTTD 1.10.1 for ...".
@orudge
Copy link
Contributor

@orudge orudge commented May 1, 2020

It'd probably be better if it just says "Windows XP and higher" or similar.

@duckfullstop
Copy link
Contributor

@duckfullstop duckfullstop commented May 5, 2020

I'll put a PR in for this in a sec, seems simple enough.

@glx22 glx22 closed this in e1c85e6 May 5, 2020
LordAro added a commit to LordAro/OpenTTD that referenced this issue May 10, 2020
… Windows version (OpenTTD#8114)

* Fix OpenTTD#8102: Update Windows installer strings to reference versions newer than 7

* Fix 2c53390: Version strings incorrectly capitalised
LordAro added a commit that referenced this issue Jun 1, 2020
…s version (#8114)

* Fix #8102: Update Windows installer strings to reference versions newer than 7

* Fix 2c53390: Version strings incorrectly capitalised
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 pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants