Skip to content
This repository has been archived by the owner on Jun 16, 2024. It is now read-only.

Commit

Permalink
fix: properly maximize windows via window decoration buttons
Browse files Browse the repository at this point in the history
Apparently pressing the maximize button doesn't actually force the
window into any specific geometry. This resulted in windows staying in their
tiled geometry while all other windows would be rearranged as if the
maximized window would be floating.

This fix makes the maximized window fill the entire working area.
  • Loading branch information
Daxtorim authored and Mikhail Zolotukhin committed Nov 24, 2021
1 parent cc27485 commit 3d9c874
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/kwinscript/engine/window.ts
Original file line number Diff line number Diff line change
Expand Up @@ -331,7 +331,11 @@ export class EngineWindowImpl implements EngineWindow {
this.log.log(["Window#commit", { state: WindowState[state] }]);
switch (state) {
case WindowState.NativeMaximized:
this.window.commit(undefined, undefined, false);
this.window.commit(
this.window.surface.workingArea,
undefined,
undefined
);
break;

case WindowState.NativeFullscreen:
Expand Down

0 comments on commit 3d9c874

Please sign in to comment.