graesslin added a comment.
In https://phabricator.kde.org/D10425#204000, @fvogt wrote: > In https://phabricator.kde.org/D10425#203972, @graesslin wrote: > > > I'm against this change. The window is set to fullscreen, but that doesn't work due to a Qt bug. > > > Is the referenced patch enough to make it work? AFAICT it would also need a change in kwaylandintegration for the decoration. A fullscreen window does not have decorations. Yes, the referenced patch is enough. > The calls added here don't conflict with other parts, they simply set what should be done by Qt already. > Doing this in addition isn't even a hack as it has no downsides. I disagree. Setting a fullscreen window to frameless is confusing and wrong for anybody who is going to read the code later on. > > >> Let's not work around Qt bugs: https://bugreports.qt.io/browse/QTBUG-63748 > > Why not? Workarounds for Qt bugs are in many places already. Yes, but it is bad practice and in this case it's not even fixing all the problems. The window is still not fullscreen, which is a problem as soon as you have other fullscreen windows. If you want to workaround let's just force to wl_shell instead of xdg_shell which doesn't expose the problem. Just set env variable QT_WAYLAND_SHELL_INTEGRATION to wl-shell REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D10425 To: fvogt, #plasma, graesslin, davidedmundson, bshah Cc: ngraham, bshah, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart