notes
notes copied to clipboard
nuttyartist
UpdaterWindow doesn't have proper Window decorations on Linux
On KDE, the regular window shows a proper border + shadow so it works fine on a white background. However, UpdaterWindow ignores the KDE Qt style and blends in to the white background.
Also the "Close" and "Update" button isn't consistent with the rest of the Qt style (it looks more like Android rather than KDE).
Fixed by #469.
@guihkx Really? Did you test it? We're still setting Qt::CustomizeWindowHint on the updater window so it will not have a native decoration. It might be nice to follow the same configuration as for the main window.
Also the "Close" and "Update" button isn't consistent with the rest of the Qt style (it looks more like Android rather than KDE).
@DesiOtaku Note that their style has been intentionally customized. There is currently no option for using the native widget style.
@bjorn It "works", but I just noticed that's maybe because frameless mode seems to be ignored altogether on Wayland now... :/
KDE Plasma
GNOME
It "works", but I just noticed that's maybe because frameless mode seems to be ignored altogether on Wayland now... :/
On the KDE screenshot I don't see the updater window (though I guess it doesn't matter, since it would indeed just have the native frame, judging by the main window).
Does that mean we need to put back the Qt::FramelessWindowHint on Linux? I can't personally test on Wayland due to using proprietary NVidia drivers.
On the KDE screenshot I don't see the update window.
Dang it, I uploaded the wrong one. But it matches GNOME's.
Regarding frameless mode not working on Wayland, could that mean we need to put back the
Qt::FramelessWindowHinton Linux?
Precisely. After I re-added that to mainwindow.cpp and updaterwindow.cpp, native decorations in frameless mode are now gone, at least:
But this bug still persists. I'll re-open it and send a pull request once after I finish some chores (but feel free to take over, if you want).