Skip to content

Commit

Permalink
Merge pull request #29752 from bruvzg/window_size_limits
Browse files Browse the repository at this point in the history
Add ability to limit maximum/minimum window size.
  • Loading branch information
akien-mga authored Jun 17, 2019
2 parents fc4b93e + b924fb9 commit 0d61fc2
Show file tree
Hide file tree
Showing 10 changed files with 291 additions and 16 deletions.
22 changes: 22 additions & 0 deletions core/bind/core_bind.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -308,6 +308,14 @@ void _OS::set_window_position(const Point2 &p_position) {
OS::get_singleton()->set_window_position(p_position);
}

Size2 _OS::get_max_window_size() const {
return OS::get_singleton()->get_max_window_size();
}

Size2 _OS::get_min_window_size() const {
return OS::get_singleton()->get_min_window_size();
}

Size2 _OS::get_window_size() const {
return OS::get_singleton()->get_window_size();
}
Expand All @@ -316,6 +324,14 @@ Size2 _OS::get_real_window_size() const {
return OS::get_singleton()->get_real_window_size();
}

void _OS::set_max_window_size(const Size2 &p_size) {
OS::get_singleton()->set_max_window_size(p_size);
}

void _OS::set_min_window_size(const Size2 &p_size) {
OS::get_singleton()->set_min_window_size(p_size);
}

void _OS::set_window_size(const Size2 &p_size) {
OS::get_singleton()->set_window_size(p_size);
}
Expand Down Expand Up @@ -1139,6 +1155,10 @@ void _OS::_bind_methods() {
ClassDB::bind_method(D_METHOD("get_window_position"), &_OS::get_window_position);
ClassDB::bind_method(D_METHOD("set_window_position", "position"), &_OS::set_window_position);
ClassDB::bind_method(D_METHOD("get_window_size"), &_OS::get_window_size);
ClassDB::bind_method(D_METHOD("get_max_window_size"), &_OS::get_max_window_size);
ClassDB::bind_method(D_METHOD("get_min_window_size"), &_OS::get_min_window_size);
ClassDB::bind_method(D_METHOD("set_max_window_size", "size"), &_OS::set_max_window_size);
ClassDB::bind_method(D_METHOD("set_min_window_size", "size"), &_OS::set_min_window_size);
ClassDB::bind_method(D_METHOD("set_window_size", "size"), &_OS::set_window_size);
ClassDB::bind_method(D_METHOD("get_window_safe_area"), &_OS::get_window_safe_area);
ClassDB::bind_method(D_METHOD("set_window_fullscreen", "enabled"), &_OS::set_window_fullscreen);
Expand Down Expand Up @@ -1284,6 +1304,8 @@ void _OS::_bind_methods() {
ADD_PROPERTY(PropertyInfo(Variant::BOOL, "vsync_enabled"), "set_use_vsync", "is_vsync_enabled");
ADD_PROPERTY(PropertyInfo(Variant::BOOL, "low_processor_usage_mode"), "set_low_processor_usage_mode", "is_in_low_processor_usage_mode");
ADD_PROPERTY(PropertyInfo(Variant::BOOL, "keep_screen_on"), "set_keep_screen_on", "is_keep_screen_on");
ADD_PROPERTY(PropertyInfo(Variant::VECTOR2, "min_window_size"), "set_min_window_size", "get_min_window_size");
ADD_PROPERTY(PropertyInfo(Variant::VECTOR2, "max_window_size"), "set_max_window_size", "get_max_window_size");
ADD_PROPERTY(PropertyInfo(Variant::INT, "screen_orientation", PROPERTY_HINT_ENUM, "Landscape,Portrait,Reverse Landscape,Reverse Portrait,Sensor Landscape,Sensor Portrait,Sensor"), "set_screen_orientation", "get_screen_orientation");
ADD_GROUP("Window", "window_");
ADD_PROPERTY(PropertyInfo(Variant::BOOL, "window_borderless"), "set_borderless_window", "get_borderless_window");
Expand Down
4 changes: 4 additions & 0 deletions core/bind/core_bind.h
Original file line number Diff line number Diff line change
Expand Up @@ -175,9 +175,13 @@ class _OS : public Object {
virtual int get_screen_dpi(int p_screen = -1) const;
virtual Point2 get_window_position() const;
virtual void set_window_position(const Point2 &p_position);
virtual Size2 get_max_window_size() const;
virtual Size2 get_min_window_size() const;
virtual Size2 get_window_size() const;
virtual Size2 get_real_window_size() const;
virtual Rect2 get_window_safe_area() const;
virtual void set_max_window_size(const Size2 &p_size);
virtual void set_min_window_size(const Size2 &p_size);
virtual void set_window_size(const Size2 &p_size);
virtual void set_window_fullscreen(bool p_enabled);
virtual bool is_window_fullscreen() const;
Expand Down
4 changes: 4 additions & 0 deletions core/os/os.h
Original file line number Diff line number Diff line change
Expand Up @@ -205,8 +205,12 @@ class OS {
virtual int get_screen_dpi(int p_screen = -1) const { return 72; }
virtual Point2 get_window_position() const { return Vector2(); }
virtual void set_window_position(const Point2 &p_position) {}
virtual Size2 get_max_window_size() const { return Size2(); };
virtual Size2 get_min_window_size() const { return Size2(); };
virtual Size2 get_window_size() const = 0;
virtual Size2 get_real_window_size() const { return get_window_size(); }
virtual void set_min_window_size(const Size2 p_size) {}
virtual void set_max_window_size(const Size2 p_size) {}
virtual void set_window_size(const Size2 p_size) {}
virtual void set_window_fullscreen(bool p_enabled) {}
virtual bool is_window_fullscreen() const { return true; }
Expand Down
6 changes: 6 additions & 0 deletions doc/classes/OS.xml
Original file line number Diff line number Diff line change
Expand Up @@ -811,6 +811,12 @@
<member name="low_processor_usage_mode" type="bool" setter="set_low_processor_usage_mode" getter="is_in_low_processor_usage_mode">
If [code]true[/code], the engine optimizes for low processor usage by only refreshing the screen if needed. Can improve battery consumption on mobile.
</member>
<member name="min_window_size" type="Vector2" setter="set_min_window_size" getter="get_min_window_size">
The minimum size of the window (without counting window manager decorations). Does not affect fullscreen mode. Set to [code](0, 0)[/code] to reset to the system default value.
</member>
<member name="max_window_size" type="Vector2" setter="set_max_window_size" getter="get_max_window_size">
The maximum size of the window (without counting window manager decorations). Does not affect fullscreen mode. Set to [code](0, 0)[/code] to reset to the system default value.
</member>
<member name="screen_orientation" type="int" setter="set_screen_orientation" getter="get_screen_orientation" enum="_OS.ScreenOrientation">
The current screen orientation.
</member>
Expand Down
7 changes: 7 additions & 0 deletions platform/osx/os_osx.h
Original file line number Diff line number Diff line change
Expand Up @@ -137,6 +137,9 @@ class OS_OSX : public OS_Unix {
String im_text;
Point2 im_selection;

Size2 min_size;
Size2 max_size;

PowerOSX *power_manager;

CrashHandler crash_handler;
Expand Down Expand Up @@ -235,6 +238,10 @@ class OS_OSX : public OS_Unix {

virtual Point2 get_window_position() const;
virtual void set_window_position(const Point2 &p_position);
virtual Size2 get_max_window_size() const;
virtual Size2 get_min_window_size() const;
virtual void set_min_window_size(const Size2 p_size);
virtual void set_max_window_size(const Size2 p_size);
virtual void set_window_size(const Size2 p_size);
virtual void set_window_fullscreen(bool p_enabled);
virtual bool is_window_fullscreen() const;
Expand Down
66 changes: 66 additions & 0 deletions platform/osx/os_osx.mm
Original file line number Diff line number Diff line change
Expand Up @@ -267,10 +267,23 @@ - (BOOL)windowShouldClose:(id)sender {

- (void)windowDidEnterFullScreen:(NSNotification *)notification {
OS_OSX::singleton->zoomed = true;

[OS_OSX::singleton->window_object setContentMinSize:NSMakeSize(0, 0)];
[OS_OSX::singleton->window_object setContentMaxSize:NSMakeSize(FLT_MAX, FLT_MAX)];
}

- (void)windowDidExitFullScreen:(NSNotification *)notification {
OS_OSX::singleton->zoomed = false;

if (OS_OSX::singleton->min_size != Size2()) {
Size2 size = OS_OSX::singleton->min_size / OS_OSX::singleton->_display_scale();
[OS_OSX::singleton->window_object setContentMinSize:NSMakeSize(size.x, size.y)];
}
if (OS_OSX::singleton->max_size != Size2()) {
Size2 size = OS_OSX::singleton->max_size / OS_OSX::singleton->_display_scale();
[OS_OSX::singleton->window_object setContentMaxSize:NSMakeSize(size.x, size.y)];
}

if (!OS_OSX::singleton->resizable)
[OS_OSX::singleton->window_object setStyleMask:[OS_OSX::singleton->window_object styleMask] & ~NSWindowStyleMaskResizable];
}
Expand Down Expand Up @@ -2357,6 +2370,46 @@ static int get_screen_index(NSScreen *screen) {
return Size2(frame.size.width, frame.size.height) * _display_scale();
}

Size2 OS_OSX::get_max_window_size() const {
return max_size;
}

Size2 OS_OSX::get_min_window_size() const {
return min_size;
}

void OS_OSX::set_min_window_size(const Size2 p_size) {

if ((p_size != Size2()) && (max_size != Size2()) && ((p_size.x > max_size.x) || (p_size.y > max_size.y))) {
WARN_PRINT("Minimum window size can't be larger than maximum window size!");
return;
}
min_size = p_size;

if ((min_size != Size2()) && !zoomed) {
Size2 size = min_size / _display_scale();
[window_object setContentMinSize:NSMakeSize(size.x, size.y)];
} else {
[window_object setContentMinSize:NSMakeSize(0, 0)];
}
}

void OS_OSX::set_max_window_size(const Size2 p_size) {

if ((p_size != Size2()) && ((p_size.x < min_size.x) || (p_size.y < min_size.y))) {
WARN_PRINT("Maximum window size can't be smaller than minimum window size!");
return;
}
max_size = p_size;

if ((max_size != Size2()) && !zoomed) {
Size2 size = max_size / _display_scale();
[window_object setContentMaxSize:NSMakeSize(size.x, size.y)];
} else {
[window_object setContentMaxSize:NSMakeSize(FLT_MAX, FLT_MAX)];
}
}

void OS_OSX::set_window_size(const Size2 p_size) {

Size2 size = p_size / _display_scale();
Expand Down Expand Up @@ -2386,6 +2439,19 @@ static int get_screen_index(NSScreen *screen) {
set_window_per_pixel_transparency_enabled(false);
if (!resizable)
[window_object setStyleMask:[window_object styleMask] | NSWindowStyleMaskResizable];
if (p_enabled) {
[window_object setContentMinSize:NSMakeSize(0, 0)];
[window_object setContentMaxSize:NSMakeSize(FLT_MAX, FLT_MAX)];
} else {
if (min_size != Size2()) {
Size2 size = min_size / _display_scale();
[window_object setContentMinSize:NSMakeSize(size.x, size.y)];
}
if (max_size != Size2()) {
Size2 size = max_size / _display_scale();
[window_object setContentMaxSize:NSMakeSize(size.x, size.y)];
}
}
[window_object toggleFullScreen:nil];
}
zoomed = p_enabled;
Expand Down
47 changes: 46 additions & 1 deletion platform/windows/os_windows.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -355,7 +355,23 @@ LRESULT OS_Windows::WndProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam)

return 0; // Return To The Message Loop
}

case WM_GETMINMAXINFO: {
if (video_mode.resizable && !video_mode.fullscreen) {
Size2 decor = get_real_window_size() - get_window_size(); // Size of window decorations
MINMAXINFO *min_max_info = (MINMAXINFO *)lParam;
if (min_size != Size2()) {
min_max_info->ptMinTrackSize.x = min_size.x + decor.x;
min_max_info->ptMinTrackSize.y = min_size.y + decor.y;
}
if (max_size != Size2()) {
min_max_info->ptMaxTrackSize.x = max_size.x + decor.x;
min_max_info->ptMaxTrackSize.y = max_size.y + decor.y;
}
return 0;
} else {
break;
}
}
case WM_PAINT:

Main::force_redraw();
Expand Down Expand Up @@ -1771,6 +1787,7 @@ void OS_Windows::set_window_position(const Point2 &p_position) {
last_pos = p_position;
update_real_mouse_position();
}

Size2 OS_Windows::get_window_size() const {

if (minimized) {
Expand All @@ -1783,6 +1800,33 @@ Size2 OS_Windows::get_window_size() const {
}
return Size2();
}

Size2 OS_Windows::get_max_window_size() const {
return max_size;
}

Size2 OS_Windows::get_min_window_size() const {
return min_size;
}

void OS_Windows::set_min_window_size(const Size2 p_size) {

if ((p_size != Size2()) && (max_size != Size2()) && ((p_size.x > max_size.x) || (p_size.y > max_size.y))) {
WARN_PRINT("Minimum window size can't be larger than maximum window size!");
return;
}
min_size = p_size;
}

void OS_Windows::set_max_window_size(const Size2 p_size) {

if ((p_size != Size2()) && ((p_size.x < min_size.x) || (p_size.y < min_size.y))) {
WARN_PRINT("Maximum window size can't be smaller than minimum window size!");
return;
}
max_size = p_size;
}

Size2 OS_Windows::get_real_window_size() const {

RECT r;
Expand All @@ -1791,6 +1835,7 @@ Size2 OS_Windows::get_real_window_size() const {
}
return Size2();
}

void OS_Windows::set_window_size(const Size2 p_size) {

int w = p_size.width;
Expand Down
7 changes: 7 additions & 0 deletions platform/windows/os_windows.h
Original file line number Diff line number Diff line change
Expand Up @@ -126,6 +126,9 @@ class OS_Windows : public OS {

HCURSOR hCursor;

Size2 min_size;
Size2 max_size;

Size2 window_rect;
VideoMode video_mode;
bool preserve_window_size = false;
Expand Down Expand Up @@ -238,6 +241,10 @@ class OS_Windows : public OS {
virtual void set_window_position(const Point2 &p_position);
virtual Size2 get_window_size() const;
virtual Size2 get_real_window_size() const;
virtual Size2 get_max_window_size() const;
virtual Size2 get_min_window_size() const;
virtual void set_min_window_size(const Size2 p_size);
virtual void set_max_window_size(const Size2 p_size);
virtual void set_window_size(const Size2 p_size);
virtual void set_window_fullscreen(bool p_enabled);
virtual bool is_window_fullscreen() const;
Expand Down
Loading

0 comments on commit 0d61fc2

Please sign in to comment.