<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>449844</bug_id>
          
          <creation_ts>2022-02-09 12:22:54 +0000</creation_ts>
          <short_desc>Fall Apart effect in new overview</short_desc>
          <delta_ts>2022-02-10 21:59:10 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Plasma</classification>
          <product>kwin</product>
          <component>effects-overview</component>
          <version>git-stable-Plasma/5.24</version>
          <rep_platform>Arch Linux</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="saxn">saileshpoudel0</reporter>
          <assigned_to name="KWin default assignee">kwin-bugs-null</assigned_to>
          <cc>nate</cc>
          
          <cf_commitlink>https://invent.kde.org/plasma/kwin/commit/75462970f344acaab5b4c661cd9a415744545935</cf_commitlink>
          <cf_versionfixedin>5.24.1</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2102355</commentid>
    <comment_count>0</comment_count>
    <who name="saxn">saileshpoudel0</who>
    <bug_when>2022-02-09 12:22:54 +0000</bug_when>
    <thetext>SUMMARY
Effect only shows when there is no overview effect.

STEPS TO REPRODUCE
1. Enable fall apart effect in Desktop Effects
2. open any two application and go to new overview effect
3. Close any window from the overview and observe the effect

OBSERVED RESULT
When the window is closed also when the overview effect is closed then only the fall apart effect is shown

EXPECTED RESULT
It should show the fall apart effect within the overview effect.

Operating System: Arch Linux
KDE Plasma Version: 5.24.0
KDE Frameworks Version: 5.90.0
Qt Version: 5.15.2
Kernel Version: 5.16.7-zen1-1-zen (64-bit)
Graphics Platform: Wayland
Processors: 8 × Intel® Core™ i5-9300H CPU @ 2.40GHz
Memory: 7.6 GiB of RAM
Graphics Processor: Mesa Intel® UHD Graphics 630</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2102370</commentid>
    <comment_count>1</comment_count>
    <who name="Vlad Zahorodnii">vlad.zahorodnii</who>
    <bug_when>2022-02-09 12:40:58 +0000</bug_when>
    <thetext>Those two effects are implemented using different infrastructure. It&apos;s surprising that the fall apart is not disabled when there&apos;s a running fullscreen effect though, like all other animation effects.

Adding fall apart integration in the overview effect would be very tedious and I&apos;m not sure that it&apos;s actually worth it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2102375</commentid>
    <comment_count>2</comment_count>
    <who name="saxn">saileshpoudel0</who>
    <bug_when>2022-02-09 12:45:20 +0000</bug_when>
    <thetext>i agree, but fall apart should not apply on overview effect then i guess.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2102383</commentid>
    <comment_count>3</comment_count>
    <who name="Bug Janitor Service">bug-janitor</who>
    <bug_when>2022-02-09 13:12:06 +0000</bug_when>
    <thetext>A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/1986</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2102462</commentid>
    <comment_count>4</comment_count>
    <who name="Vlad Zahorodnii">vlad.zahorodnii</who>
    <bug_when>2022-02-09 15:46:52 +0000</bug_when>
    <thetext>Git commit e419faee39b23d4b1b9f4359b07d31b5e95e7e6e by Vlad Zahorodnii.
Committed on 09/02/2022 at 13:09.
Pushed by vladz into branch &apos;master&apos;.

effects/fallapart: Avoid animating windows while there&apos;s a fullscreen effect

While there&apos;s a fullscreen effect, the fall apart effect should avoid
animating windows as it can corrupt or interfere with the active full
screen effect. This matches behavior of many other animation effects in
kwin.

M  +2    -0    src/effects/fallapart/fallapart.cpp

https://invent.kde.org/plasma/kwin/commit/e419faee39b23d4b1b9f4359b07d31b5e95e7e6e</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2102463</commentid>
    <comment_count>5</comment_count>
    <who name="Vlad Zahorodnii">vlad.zahorodnii</who>
    <bug_when>2022-02-09 15:47:20 +0000</bug_when>
    <thetext>Git commit 75462970f344acaab5b4c661cd9a415744545935 by Vlad Zahorodnii.
Committed on 09/02/2022 at 15:47.
Pushed by vladz into branch &apos;Plasma/5.24&apos;.

effects/fallapart: Avoid animating windows while there&apos;s a fullscreen effect

While there&apos;s a fullscreen effect, the fall apart effect should avoid
animating windows as it can corrupt or interfere with the active full
screen effect. This matches behavior of many other animation effects in
kwin.


(cherry picked from commit e419faee39b23d4b1b9f4359b07d31b5e95e7e6e)

M  +2    -0    src/effects/fallapart/fallapart.cpp

https://invent.kde.org/plasma/kwin/commit/75462970f344acaab5b4c661cd9a415744545935</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>