Steps 1. Maximise the window using the Plasma window controls 2. Enter Full Screen Mode 3. Exit Full Screen Mode Expected 1. Elisa should return to the Maximised state Actual 1. Elisa returns to a irrelevant non-maximised position
Can confirm. Will investigate, but I think there's a chance this is actually a KWin bug.
Ah no, it's an Elisa bug. When exiting full screen, we run showNormal() which explicitly makes it windowed. We need to keep track of whether the prior state was windowed or maximized, and if it was maximized, do showMaximized() instead.
A possibly relevant merge request was started @ https://invent.kde.org/multimedia/elisa/-/merge_requests/423
Thank you!
You're welcome!
Git commit b5247cc21289759384f18422e36d83b9a41e2dff by Nate Graham. Committed on 16/03/2023 at 15:04. Pushed by ngraham into branch 'master'. Restore prior state correctly when leaving full screen mode Previously we would always make the window windowed when leaving full screen mode. But this isn't the only possible state for the window to be in at the moment it entered full screen mode. Let's handle all of those states properly. FIXED-IN: 23.04 M +1 -1 src/qml/ApplicationMenu.qml M +31 -0 src/qml/ElisaMainWindow.qml M +1 -1 src/qml/HeaderBar.qml M +1 -1 src/qml/NativeGlobalMenu.qml https://invent.kde.org/multimedia/elisa/commit/b5247cc21289759384f18422e36d83b9a41e2dff
Git commit 58e3ab100e36271dca5700edccec5b30e8463c75 by Nate Graham. Committed on 16/03/2023 at 15:28. Pushed by ngraham into branch 'release/23.04'. Restore prior state correctly when leaving full screen mode Previously we would always make the window windowed when leaving full screen mode. But this isn't the only possible state for the window to be in at the moment it entered full screen mode. Let's handle all of those states properly. FIXED-IN: 23.04 (cherry picked from commit b5247cc21289759384f18422e36d83b9a41e2dff) M +1 -1 src/qml/ApplicationMenu.qml M +31 -0 src/qml/ElisaMainWindow.qml M +1 -1 src/qml/HeaderBar.qml M +1 -1 src/qml/NativeGlobalMenu.qml https://invent.kde.org/multimedia/elisa/commit/58e3ab100e36271dca5700edccec5b30e8463c75