Review Request 127216: [KStatusNotifierItem] MinimizeRestore does not "run" over the desktop on X11

[Anthony Fieroni]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127216/#review94623
-----------------------------------------------------------



So as Thomas says in https://git.reviewboard.kde.org/r/127631/ it's a Qt issue, options are:
1. leave the code as is (i'm not prefer it, the bug is annoying)
2. do it on *right* way storing / restoring the position and when Qt fix the issue it will *just* work till then windows will be shown depend on KWin placement policy. Since this policy is user selectable it's not big problem, about me (i prefer this way)
3. Someone can test this patch with more recent version of Qt 5.6.1 or 5.7

- Anthony Fieroni


On Април 10, 2016, 10:48 след обяд, Anthony Fieroni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127216/
> -----------------------------------------------------------
> 
> (Updated Април 10, 2016, 10:48 след обяд)
> 
> 
> Review request for KDE Frameworks, Martin Gräßlin and Martin Klapetek.
> 
> 
> Bugs: 356523
>     https://bugs.kde.org/show_bug.cgi?id=356523
> 
> 
> Repository: knotifications
> 
> 
> Description
> -------
> 
> Store position of widget before hide it
> 
> 
> Diffs
> -----
> 
>   src/kstatusnotifieritem.cpp 27abfb8 
> 
> Diff: https://git.reviewboard.kde.org/r/127216/diff/
> 
> 
> Testing
> -------
> 
> Tested on pixel ratio = 1 with Amarok, Kmail, Akregator, Kalarm, Ktimer
> Close with 'X' - restore in correct pos
> Hide by click at icon in systray - restore correct pos
> 
> 
> Thanks,
> 
> Anthony Fieroni
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160415/1e7bc77c/attachment.html>