Bug 466177

Summary: Magic Lamp effect should not go below floating panel
Product: [Plasma] kwin Reporter: Bill <bill73653>
Component: effects-variousAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: minor CC: nate, niccolo.venerandi
Priority: NOR    
Version: 5.27.0   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 5.27.6
Attachments: Video of Magic lamp going past the panel.

Description Bill 2023-02-21 02:09:32 UTC
Created attachment 156560 [details]
Video of Magic lamp going past the panel.

SUMMARY
Magic Lamp effect goes offscreen, past the floating panel.  

STEPS TO REPRODUCE
1. Enable "magic lamp" desktop effect
2.  Enable floating panels
3.  Activate the magic lamp effect.    

OBSERVED RESULT
The application does not get sucked into the apps position, and instead goes off screen, below where the app is positioned.  

EXPECTED RESULT

The magic lamp effect should not go below the floating panel.  

SOFTWARE/OS VERSIONS

KDE Plasma Version:  5.27
KDE Frameworks Version: 103
Qt Version:  5.15.8

ADDITIONAL INFORMATION

This could be a matter of personal preference, but I think the magic lamp should go into the application icon on the taskbar, instead of going into the bottom of the screen below the floating panel.  Video attached.
Comment 1 Vlad Zahorodnii 2023-05-16 06:17:25 UTC
Git commit 00a1a506d18f99c16451af2fd5ee859c646b5b14 by Vlad Zahorodnii.
Committed on 16/05/2023 at 06:03.
Pushed by vladz into branch 'master'.

effects/magiclamp: Make it look good with floating panels

The magic lamp literally "squashes" the window through the window icon
in the task manager.

It's assumed that there's nothing below the panel, so the magic lamp
doesn't perform any clipping.

With floating panels, it's not the case. So let's clamp the x or the y
coordinates when the window moves horizontally or vertically,
respectively, in order to ensure that the window is not visible in the
gap between the floating panel and the screen edge.
Related: bug 361121

M  +68   -8    src/plugins/magiclamp/magiclamp.cpp

https://invent.kde.org/plasma/kwin/commit/00a1a506d18f99c16451af2fd5ee859c646b5b14
Comment 2 Vlad Zahorodnii 2023-05-16 07:12:09 UTC
Git commit e749d285feccc91111255155b2ee2d21cb00b205 by Vlad Zahorodnii.
Committed on 16/05/2023 at 06:25.
Pushed by vladz into branch 'Plasma/5.27'.

effects/magiclamp: Make it look good with floating panels

The magic lamp literally "squashes" the window through the window icon
in the task manager.

It's assumed that there's nothing below the panel, so the magic lamp
doesn't perform any clipping.

With floating panels, it's not the case. So let's clamp the x or the y
coordinates when the window moves horizontally or vertically,
respectively, in order to ensure that the window is not visible in the
gap between the floating panel and the screen edge.
Related: bug 361121
(cherry picked from commit 00a1a506d18f99c16451af2fd5ee859c646b5b14)

M  +68   -8    src/effects/magiclamp/magiclamp.cpp

https://invent.kde.org/plasma/kwin/commit/e749d285feccc91111255155b2ee2d21cb00b205