Add ClydeWindow API to set borderless mode#6618
Conversation
|
Incredibly based PR. |
PowerfulBacon
changed the title
| bool IsFocused { get; } | ||
| bool IsMinimized { get; } | ||
| bool IsVisible { get; set; } | ||
| bool IsTitleBarVisible { get; set; } |
There was a problem hiding this comment.
This probably shouldn't be named this given it changes more than the title bar?
There was a problem hiding this comment.
Thanks for the review, I'll update the names of everything to be centered around the 'border' rather than the 'title bar' since thats how SDL names it.
There was a problem hiding this comment.
I have updated it to be IsBordered rather than IsTitleBarVisible, as that's the naming convention that SDL uses
|
We had a discussion on the SS14 discord, since wayland doesn't support setting the window position then we probably want to make it so that wayland windows cannot be made borderless. The question is whether or not that should be added in this PR, or in a PR that adds the ability to set window positions, since this PR's utility is a lot lower without the ability to set position at runtime. |
3 participants
Adds support for borderless windows in the Clyde window engine. A window can be set to borderless by calling the
SetWindowTitleBarVisible.A decision needs to be made as to whether I should rename this method to
SetWindowBorderModeor something similar.This is required for fancy TGUI window support in OpenDream: