fix: no crashes in closing transients -- might fix others too; ref #110, #109

This commit is contained in:
Derek Stevens 2021-10-14 22:32:20 -06:00 committed by Mikhail Zolotukhin
parent 49ac131044
commit 502f812081
2 changed files with 38 additions and 31 deletions

View File

@ -243,19 +243,17 @@ export class TilingController implements Controller {
this.engine.arrange();
// Switch to next window if monocle with config.monocleMinimizeRest
try {
if (!this.currentWindow && this.engine.isLayoutMonocleAndMinimizeRest()) {
this.engine.focusOrder(1, true);
/* HACK: force window to maximize if it isn't already
* This is ultimately to trigger onWindowFocused() at the right time
*/
this.engine.focusOrder(1, true);
this.engine.focusOrder(-1, true);
}
} catch {
/* HACK for the HACK: transient modals cause an error with the above workaround,
* so if we catch it here and ignore it, all is well */
return;
if (
!window.window.isDialog &&
!this.currentWindow &&
this.engine.isLayoutMonocleAndMinimizeRest()
) {
this.engine.focusOrder(1, true);
/* HACK: force window to maximize if it isn't already
* This is ultimately to trigger onWindowFocused() at the right time
*/
this.engine.focusOrder(1, true);
this.engine.focusOrder(-1, true);
}
}
@ -352,25 +350,29 @@ export class TilingController implements Controller {
}
public onWindowFocused(window: Window): void {
window.timestamp = new Date().getTime();
this.currentWindow = window;
// Minimize other windows if Monocle and config.monocleMinimizeRest
if (
this.engine.isLayoutMonocleAndMinimizeRest() &&
this.engine.windows.getVisibleTiles(window.surface).includes(window)
) {
/* If a window hasn't been focused in this layout yet, ensure its geometry
* gets maximized.
*/
this.engine
.currentLayoutOnCurrentSurface()
.apply(
this,
this.engine.windows.getAllTileables(window.surface),
window.surface.workingArea
);
try {
window.timestamp = new Date().getTime();
this.currentWindow = window;
// Minimize other windows if Monocle and config.monocleMinimizeRest
if (
this.engine.isLayoutMonocleAndMinimizeRest() &&
this.engine.windows.getVisibleTiles(window.surface).includes(window)
) {
/* If a window hasn't been focused in this layout yet, ensure its geometry
* gets maximized.
*/
this.engine
.currentLayoutOnCurrentSurface()
.apply(
this,
this.engine.windows.getAllTileables(window.surface),
window.surface.workingArea
);
this.engine.minimizeOthers(window);
this.engine.minimizeOthers(window);
}
} catch {
return;
}
}

View File

@ -20,6 +20,7 @@ export interface DriverWindow {
readonly shouldFloat: boolean;
readonly screen: number;
readonly active: boolean;
readonly isDialog: boolean;
surface: DriverSurface;
minimized: boolean;
shaded: boolean;
@ -261,4 +262,8 @@ export class KWinWindow implements DriverWindow {
return new Rect(geometry.x, geometry.y, width, height);
}
public get isDialog(): boolean {
return this.client.dialog;
}
}