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.

Reply via email to