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

(Updated Фев. 6, 2016, 4:57 след обяд)


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


Changes
-------

I try to remove netwm and X11 specific code, but i'm stuck on moving window by 
WId. I read that on Wayland is not available (on Martin's bog :)).


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 (updated)
-----

  applets/activitypager/plugin/activitypager.h d33e81e 
  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

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to