kwin/convergentwindows: Don't set maximize property for fullscreen windows

Followup to https://invent.kde.org/teams/plasma-mobile/issues/-/issues/256

Otherwise the positioning affects shell windows too
This commit is contained in:
Devin Lin 2024-03-04 14:30:37 -05:00
parent b553850e0a
commit a748ecda5b

View file

@ -13,6 +13,7 @@ Loader {
if (ShellSettings.Settings.convergenceModeEnabled) { if (ShellSettings.Settings.convergenceModeEnabled) {
window.noBorder = false; window.noBorder = false;
} else { } else {
if (!window.fullScreen) {
const output = window.output; const output = window.output;
const desktop = window.desktops[0]; // assume it's the first desktop that the window is on const desktop = window.desktops[0]; // assume it's the first desktop that the window is on
const maximizeRect = KWinComponents.Workspace.clientArea(KWinComponents.Workspace.MaximizeArea, output, desktop); const maximizeRect = KWinComponents.Workspace.clientArea(KWinComponents.Workspace.MaximizeArea, output, desktop);
@ -21,14 +22,17 @@ Loader {
// between maximizing and window decorations being turned off (changing window height) // between maximizing and window decorations being turned off (changing window height)
// see: https://invent.kde.org/teams/plasma-mobile/issues/-/issues/256 // see: https://invent.kde.org/teams/plasma-mobile/issues/-/issues/256
window.frameGeometry = maximizeRect; window.frameGeometry = maximizeRect;
}
// turn off window decorations // turn off window decorations
window.noBorder = true; window.noBorder = true;
if (!window.fullScreen) {
// run maximize after to ensure the state is maximized // run maximize after to ensure the state is maximized
window.setMaximize(true, true); window.setMaximize(true, true);
} }
} }
}
Connections { Connections {
target: ShellSettings.Settings target: ShellSettings.Settings