Bug 250856 - JJ: Use a bit of Wobbly effect while closing a window also.
Summary: JJ: Use a bit of Wobbly effect while closing a window also.
Status: RESOLVED INTENTIONAL
Alias: None
Product: kwin
Classification: Plasma
Component: effects-window-management (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR wishlist
Target Milestone: 4.11
Assignee: KWin default assignee
URL:
Keywords: junior-jobs
Depends on:
Blocks:
 
Reported: 2010-09-11 11:14 UTC by Yogesh Marwaha
Modified: 2023-01-25 12:51 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Yogesh Marwaha 2010-09-11 11:14:50 UTC
Version:           unspecified (using KDE 4.5.1) 
OS:                Linux

Use wobbly effect while closing a window, presently it seem not to be the case.

Reproducible: Didn't try
Comment 1 Martin Flöser 2010-09-11 11:19:47 UTC
Could you please describe what you want in a more clear way? I do not get at 
all what you want to have.
Comment 2 Yogesh Marwaha 2010-09-11 11:43:31 UTC
I want (as an option, may be) that the wobbly effect applied to a window being shown (i.e. while its appearance) is also repeated (in some form) for a window being closed (i.e. while its disappearance).
Comment 3 Martin Flöser 2011-03-19 10:53:48 UTC
just needs to implement the windowClosed slot and ref/unref the window.
Comment 4 Alexander Jones 2012-10-16 17:29:03 UTC
Could changing part of line 506 in wobblywindows.cpp from "w->data(WindowAddedGrabRole).value<void*>() != this" to "w->data(WindowAddedGrabRole).value<void*>() == this" work?
Comment 5 Thomas Lübking 2012-10-16 17:40:47 UTC
Hardly. Esp. since the query seems completely pointless since the dataRole is apparently never set to "this" (not to mention that it's the "added" role)
Comment 6 Vlad Zahorodnii 2023-01-25 12:51:12 UTC
EffectsHandler::windowClosed() handling had been removed a while ago. Also, this feature request has been open for longer than a decade, which is good indicator that it's unlikely that it will be implemented. If somebody wants this feature, create a MR.