Skip to content
  • David Edmundson's avatar
    [effects/slideback] Port away from deprecated EffectWindow::desktop · fe47a8d9
    David Edmundson authored
    Summary:
    Not only does the function windowsShareDesktop become a lot more complex
    with the new desktop cardinality comparing if they share any desktop
    isn't actually what we want.
    
    If I have 2 windows on desktop 1 and the lower stacked window is also on
    desktop 2, when I switch desktops the focus will change, but we don't
    want to animate anything here as on this dekstop it has nothing to slide
    in front of.
    
    Instead this patch simply checks both windows are on the current
    desktop.
    
    Test Plan:
    Loaded a few windows. Some on one, some on more.
    Before if a window was on desktop 1&2 it wouldn't animate on 2.
    
    Windows now seem to animate when they should and not when they shouldn't.
    
    Reviewers: #kwin, zzag
    
    Reviewed By: #kwin, zzag
    
    Subscribers: kwin
    
    Tags: #kwin
    
    Differential Revision: https://phabricator.kde.org/D17943
    fe47a8d9