https://bugs.kde.org/show_bug.cgi?id=195046
Peter Mühlenpfordt <de...@ukn8.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Latest Commit| |https://commits.kde.org/gwe | |nview/0d917337122fcc0227058 | |6effa99997df881a32d Resolution|--- |FIXED Status|CONFIRMED |RESOLVED --- Comment #10 from Peter Mühlenpfordt <de...@ukn8.de> --- Git commit 0d917337122fcc02270586effa99997df881a32d by Peter Mühlenpfordt. Committed on 09/02/2018 at 12:40. Pushed by muhlenpfordt into branch 'master'. Keep Gwenview fullscreen mode in sync with window manager Summary: Toggling fullscreen mode in Gwenview does not reflect in window manager and vice versa. This patch changes toggling fullscreen mode from `setWindowState()` to `KToggleFullScreenAction::setFullScreen()` as the api doc recommends. Test Plan: Start Gwenview in windowed or fullscreen mode and toggle by window manager, e.g. {nav Alt+F3 > More Actions > Fullscreen}. Use other tools to toggle Gwenview's fullscreen mode: ``` kstart --fullscreen gwenview image.jpg wmctrl -i -r <WINID> -b add,fullscreen wmctrl -i -r <WINID> -b remove,fullscreen ``` Reviewers: rkflx, ngraham Reviewed By: rkflx Subscribers: broulik Differential Revision: https://phabricator.kde.org/D10360 M +9 -10 app/mainwindow.cpp https://commits.kde.org/gwenview/0d917337122fcc02270586effa99997df881a32d -- You are receiving this mail because: You are watching all bug changes.