0011122: Bad resizing on small screens (fullscreen mode)

(0031691)
advil (administrator)
2017-06-25 10:13

Crawl should run just fine as windowed at 1366x768, and does so at least on a mac (macbook air 11"). Can you confirm that you have

tile_full_screen = false

in your rc file?

I'm not sure offhand why it's doing the -90 thing, that could be a bug (maybe intended for non-fullscreen), but it looks like as a workaround you should be able to override that by setting:

tile_window_width = 0
tile_window_height = 0

(0031692)
advil (administrator)
2017-06-25 10:33
edited on: 2017-06-25 11:02

Just a bit more info, the size below which fullscreen is forced is 1024x800 (in either dimension). I think though that if `tile_full_screen=false`, it won't do that -- it would need to be set as auto.

Also, on a mac setting the windowsize in fullscreen mode has no effect [edit: setting it negatively, i.e. probably setting it to a non-allowed resolution], so there may be some platform differences here that are causing problems.

(0031693)
Fixer (reporter)
2017-06-25 13:31
edited on: 2017-06-25 13:31

I've used clean config, options state that "tile_full_screen = false", I believe it properly detected that 768px < 800px and went fullscreen and used
tile_window_width = -90
tile_window_height = -90
which resulted in ugly downscaling in fullscreen mode to 1276x678. Even windows warned me about this unoptimal resolution.
Setting tile_window_width and tile_window_height to zero made it run properly in fullscreen native resolution.

(0031694)
Fixer (reporter)
2017-06-25 13:45
edited on: 2017-06-25 13:46

Fullscreen mode now uses native resolution and ignores tile_window_width/height setting since https://github.com/crawl/crawl/commit/b3e98cc55699 [^]

Bug report can be closed I guess.

(0031695)
advil (administrator)
2017-06-25 18:34

'options state that "tile_full_screen = false"'

This is actually incorrect, it is auto by default (but I guess the docs never got updated because there's no way to explicitly set auto except leaving the field out). If you explicitly set it to false it will override the auto mode.

(0031696)
Fixer (reporter)
2017-06-26 20:47
edited on: 2017-06-26 20:49

I've tried new build on 1366*768 screen and it runs with proper native resolution in fullscreen mode. Bug is fixed for me. Thanks!

Can be closed.

(0031881)
advil (administrator)
2017-10-12 18:36

belatedly marking this as resolved, I think this is entirely fixed (including the docs).

Notes
(0031691) advil (administrator) 2017-06-25 10:13	Crawl should run just fine as windowed at 1366x768, and does so at least on a mac (macbook air 11"). Can you confirm that you have tile_full_screen = false in your rc file? I'm not sure offhand why it's doing the -90 thing, that could be a bug (maybe intended for non-fullscreen), but it looks like as a workaround you should be able to override that by setting: tile_window_width = 0 tile_window_height = 0

(0031692) advil (administrator) 2017-06-25 10:33 edited on: 2017-06-25 11:02	Just a bit more info, the size below which fullscreen is forced is 1024x800 (in either dimension). I think though that if `tile_full_screen=false`, it won't do that -- it would need to be set as auto. Also, on a mac setting the windowsize in fullscreen mode has no effect [edit: setting it negatively, i.e. probably setting it to a non-allowed resolution], so there may be some platform differences here that are causing problems.

(0031693) Fixer (reporter) 2017-06-25 13:31 edited on: 2017-06-25 13:31	I've used clean config, options state that "tile_full_screen = false", I believe it properly detected that 768px < 800px and went fullscreen and used tile_window_width = -90 tile_window_height = -90 which resulted in ugly downscaling in fullscreen mode to 1276x678. Even windows warned me about this unoptimal resolution. Setting tile_window_width and tile_window_height to zero made it run properly in fullscreen native resolution.

(0031694) Fixer (reporter) 2017-06-25 13:45 edited on: 2017-06-25 13:46	Fullscreen mode now uses native resolution and ignores tile_window_width/height setting since https://github.com/crawl/crawl/commit/b3e98cc55699 [^] Bug report can be closed I guess.

(0031695) advil (administrator) 2017-06-25 18:34	'options state that "tile_full_screen = false"' This is actually incorrect, it is auto by default (but I guess the docs never got updated because there's no way to explicitly set auto except leaving the field out). If you explicitly set it to false it will override the auto mode.

(0031696) Fixer (reporter) 2017-06-26 20:47 edited on: 2017-06-26 20:49	I've tried new build on 1366*768 screen and it runs with proper native resolution in fullscreen mode. Bug is fixed for me. Thanks! Can be closed.

(0031881) advil (administrator) 2017-10-12 18:36	belatedly marking this as resolved, I think this is entirely fixed (including the docs).

Issue History
Date Modified	Username	Field	Change
2017-06-25 00:59	Fixer	New Issue
2017-06-25 01:01	Fixer	Issue Monitored: Fixer
2017-06-25 10:13	advil	Note Added: 0031691
2017-06-25 10:33	advil	Note Added: 0031692
2017-06-25 11:02	advil	Note Edited: 0031692
2017-06-25 13:31	Fixer	Note Added: 0031693
2017-06-25 13:31	Fixer	Note Edited: 0031693
2017-06-25 13:45	Fixer	Note Added: 0031694
2017-06-25 13:46	Fixer	Note Edited: 0031694
2017-06-25 13:46	Fixer	Issue End Monitor: Fixer
2017-06-25 18:34	advil	Note Added: 0031695
2017-06-26 20:47	Fixer	Note Added: 0031696
2017-06-26 20:48	Fixer	Note Edited: 0031696
2017-06-26 20:49	Fixer	Note Edited: 0031696
2017-10-12 18:36	advil	Note Added: 0031881
2017-10-12 18:36	advil	Status	new => resolved
2017-10-12 18:36	advil	Fixed in Branch	=> 0.21 development branch
2017-10-12 18:36	advil	Resolution	open => done
2017-10-12 18:36	advil	Assigned To	=> advil

Relationships