From 7238199c8455b42bd52369c04b6496f242dc581c Mon Sep 17 00:00:00 2001 From: David Edmundson Date: Wed, 12 Apr 2023 22:55:32 +0100 Subject: [PATCH] Client: Remove check for BypassWindowManagerHint when setting viewport This check was put in initially to leave a code path for clients to opt-out of Qt controlled viewport management. In practice, many X11 applications have this flag set for other purposes and this does more harm than good. Given this was never a documented feature, and it won't have been used in practice given fractional scaling is not heavily deployed outside a few linux desktops it is safe to remove. The highDpiScaleFactorRoundingPolicy remains a way to opt-out. Pick-to: 6.5 Change-Id: I2d111a76695dec27e9ee1a2a233252b70379a329 Reviewed-by: Eskil Abrahamsen Blomfeldt Reviewed-by: Vlad Zahorodnii --- src/plugins/platforms/wayland/qwaylandwindow.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/plugins/platforms/wayland/qwaylandwindow.cpp b/src/plugins/platforms/wayland/qwaylandwindow.cpp index e3070c9363a..d4a499cce11 100644 --- a/src/plugins/platforms/wayland/qwaylandwindow.cpp +++ b/src/plugins/platforms/wayland/qwaylandwindow.cpp @@ -186,7 +186,7 @@ void QWaylandWindow::initWindow() } } - if (display()->viewporter() && !window()->flags().testFlag(Qt::BypassWindowManagerHint)) { + if (display()->viewporter()) { mViewport.reset(new QWaylandViewport(display()->createViewport(this))); }