Review Request 126946: ShowDesktop is not synchronized

[Anthony Fieroni]

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

Review request for Plasma, David Edmundson, Martin Gräßlin, and Marco Martin.


Repository: kdeplasma-addons


Description
-------

Activity pager settings -> Selecting current desktop -> showDesktop. If desktop is showed by clicking on widget, it can be restored by clicking on item in task manager, which lead m_desktopDown to store incorrect state.


Diffs
-----

  applets/activitypager/plugin/activitypager.cpp b991bac 

Diff: https://git.reviewboard.kde.org/r/126946/diff/


Testing
-------

Now works correctly, but i'm affraid about checks
#if HAVE_X11
    if (!m_isX11) {
        return;
    }
why the is performed and even why the hell NETRootInfo is used in that way. And NETRootInfo::showingDesktop() about be is hell not working function.


Thanks,

Anthony Fieroni

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160201/59beec2c/attachment.html>