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

Patch: Command-line fullscreen override #6483

Closed
DorpsGek opened this issue Jun 27, 2016 · 3 comments
Closed

Patch: Command-line fullscreen override #6483

DorpsGek opened this issue Jun 27, 2016 · 3 comments
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay

Comments

@DorpsGek
Copy link
Member

MacD83 opened the ticket and wrote:

Hi devs,

This patch adds a command-line parameter to override the fullscreen settings.

It is useful for debugging, for example, in Visual Studio I can set it up to start in windowed mode. Then it doesn't interfere with my normal play settings. Some people prefer windowed mode only when playing multi-player and not single-player, and want to use links to start it windowed instead of fullscreen [1].

I have added help text that gets shown with -h. If you want to start in 800x600 windowed mode you do:

openttd.exe -r 800x600 -F false

If you instead want to start in fullscreen mode you do:

openttd.exe -F true

I chose to use -F instead of -f because it was already taken.

This is my first patch for OpenTTD, so please let me know if I need to change anything.

Thank you,
Michael Mc Donnell

[1] https://www.tt-forums.net/viewtopic.php?t=19295

Reported version: trunk
Operating system: All


This issue was imported from FlySpray: https://bugs.openttd.org/task/6483
@DorpsGek
Copy link
Member Author

adf88 wrote:

You could also just have two separate config files, one with fullscreen and one windowed, and use the '-c config_file' option.


This comment was imported from FlySpray: https://bugs.openttd.org/task/6483#comment14228

@DorpsGek
Copy link
Member Author

MacD83 wrote:

Thanks adf88! That is a good point about the config file. I will just use that for now. I wonder if anyone would still be interested in also having it as a command-line option?


This comment was imported from FlySpray: https://bugs.openttd.org/task/6483#comment14229

@DorpsGek
Copy link
Member Author

andythenorth closed the ticket.

Reason for closing: Won't implement

Flyspray clean up: an acceptable alternative solution was provided for this. I can see that this would be useful, but it's not a current goal for OpenTTD (as the alternative method exists). If this offends, discuss with andythenorth in irc :)

Thanks for the suggestion and the patch.


This comment was imported from FlySpray: https://bugs.openttd.org/task/6483

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay labels Apr 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay
Projects
None yet
Development

No branches or pull requests

1 participant