yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

wayland: Move or resize xdg-toplevel window during initialization only if geometry is enforced by a window rule

Browse source 
move() and resize() functions are not convenience helpers around the
moveResize() function. They communicate what kwin wants to see after the
corresponding change is applied. It's needed to make asynchronous
geometry updates work.

This change replaces a moveResize() during XdgToplevelClient
initialization with explicit move() and resize() function calls instead,
so it's more clear what the expected end result is.
This commit is contained in:
Vlad Zahorodnii 2021-12-14 11:06:39 +02:00
parent fed45422cf
commit b26227348c
1 changed files with 10 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/xdgshellclient.cpp
View file

@ -1367,7 +1367,16 @@ void XdgToplevelClient::initialize()
setupWindowRules(false);
moveResize(rules()->checkGeometry(frameGeometry(), true));
// Move or resize the window only if enforced by a window rule.
const QPoint forcedPosition = rules()->checkPosition(invalidPoint, true);
if (forcedPosition != invalidPoint) {
move(forcedPosition);
}
const QSize forcedSize = rules()->checkSize(QSize(), true);
if (forcedSize.isValid()) {
resize(forcedSize);
}
maximize(rules()->checkMaximize(initialMaximizeMode(), true));
setFullScreen(rules()->checkFullScreen(initialFullScreenMode(), true), false);
setOnActivities(rules()->checkActivity(activities(), true));