| Summary: | Filter isn't reset after effect ends, thus e.g. breaking it for choosing grouped windows | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | Christian (Fuchs) <kde> |
| Component: | effects-present-windows | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED DUPLICATE | ||
| Severity: | major | CC: | nate, notify |
| Priority: | NOR | ||
| Version: | 5.25.1 | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
| Attachments: |
Current behaviour, notice that on click on the taskmanager the user gets presented with a blurred screen of nothing
Suggested behaviour, effect works as expected on re-activating it |
||
|
Description
Christian (Fuchs)
2022-06-22 12:40:51 UTC
A workaround (or maybe even fix) is to simply
--- main.qml.orig 2022-06-22 14:55:29.527338810 +0200
+++ main.qml 2022-06-22 14:54:58.916537416 +0200
@@ -28,6 +28,7 @@
function start() {
container.animationEnabled = true;
container.organized = true;
+ effect.searchText = ""
}
function stop() {
but since I don't know what else might trigger that effect, this might come with side effects. Imho the correct behaviour, UX wise, would be to always clear the filter when the effect is freshly started though, no matter what triggered this. I'll poke VDG folk about it.
Created attachment 150049 [details]
Current behaviour, notice that on click on the taskmanager the user gets presented with a blurred screen of nothing
Created attachment 150050 [details]
Suggested behaviour, effect works as expected on re-activating it
This is already fixed as part of a merge request that is going to be merged in a couple of hours: https://invent.kde.org/plasma/kwin/-/merge_requests/2562 Oh, aw, and I was currently setting up kdesrc-build, next time, then. Thanks for the heads up (In reply to Christian (Fuchs) from comment #5) > Oh, aw, and I was currently setting up kdesrc-build, next time, then. > > Thanks for the heads up Didn't see this bug report earlier, just noticed that it wasn't clearing while working on it yesterday, so I thought I might aswell fix it with one line. (In reply to Niklas Stephanblome from comment #6) > (In reply to Christian (Fuchs) from comment #5) > > Oh, aw, and I was currently setting up kdesrc-build, next time, then. > > > > Thanks for the heads up > > Didn't see this bug report earlier, just noticed that it wasn't clearing > while working on it yesterday, so I thought I might aswell fix it with one > line. That's perfectly fine, I'm sure I'll find other simple things to start work on. Sidenote: probably less simple and thus not for me, but probably also a candidate for the current effort, if not fixed already: https://bugs.kde.org/show_bug.cgi?id=455783 Thanks for your work :) > Sidenote: probably less simple and thus not for me, but probably also a
> candidate for the current effort, if not fixed already:
> https://bugs.kde.org/show_bug.cgi?id=455783
>
> Thanks for your work :)
I'll leave the offer up to anyone else because I don't use two monitors (I have an ultrawide monitor instead). But maybe give it a try :) I had to work my way into the codebase too (as it got a complete QML rewrite in 5.25 and I didn't really know QML...)
*** This bug has been marked as a duplicate of bug 455753 *** |