Window.MaxWidth default at 4000

1

I was quite puzzled that whenever I go FullScreen on 4k display, there’s a border on the right side. I typically test on lower, windowed resolutions so I didn’t pay much attention to it so far. Apparently, the TCastleWindow has a default value MaxWidth = 4000, same for MaxHeight. I know it’s described in documentation but I never knew I’d have to check the meaning of the Width property :laughing:

The 4k displays are 4096 pixels, so it’s just little bit more than the default, and now I know I have to change it for every new app. But if it isn’t too much to ask for, could that value be changed in the CGE source (CastleWindow.pas) to better reflect modern display capabilities? I know you’re busy with other TODOs, so I don’t push, it’s just a small QoL thing anyway.

2

Ha, nice catch, thank you :slight_smile: I changed it to 1000 * 1000 now ( Increase default TCastleWindow.MaxWidth/Height to be million (1000 * … · castle-engine/castle-engine@85a85d6 · GitHub ).

I set this limit “4000” many years ago, probably thinking “4000 is well beyond the possible resolution of any existing monitor” :slight_smile: 4k displays didn’t exist back then probably, or I didn’t know about them. ( The whole reason why we need a non-zero number there at all is internal, we should instead allow “0” there to mean “no limit”, it would be cleaner. Some day I shall improve it. )

1 Like