| Summary: | Screen corners can't trigger new Desktop Grid Effect on Plasma 5.25 beta | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | Patrick Silva <bugseforuns> |
| Component: | effects-desktop-grid | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | jr, nate, notmart |
| Priority: | NOR | Keywords: | regression |
| Version: | 5.24.90 | ||
| Target Milestone: | --- | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/kwin/commit/4c4cb446788269df0b8f1fcba15431d4ca33a146 | Version Fixed In: | 5.25 |
| Sentry Crash Report: | |||
|
Description
Patrick Silva
2022-05-20 17:57:01 UTC
Same on X11. This sounds like a packaging bug, as the code for the old effect was fully removed in KWin's source code repo. Can you follow up with the Arch folks about it? I don't think so. I have the same problem on neon unstable. I'm at a loss, then. It's working for me. CCing Jonathan, the Plasma release manager. Maybe there's some issue with the tarballs or something? I have seen a screenshot in Vlad's blog, the new effect is almost identical to the old one. https://blog.vladzahorodnii.com/2022/05/09/whats-cooking-in-kwin-plasma-5-25/ Now I think I'm seeing the new effect on my systems, but it is not triggered via screen edges/corners. Ok cool. I had an issue early in the review process with the default top-left screen corner not working, but that is working for me now. Does it work if you re-assign it to any other screen corner? Does it wok if you click the "Defaults" button and then Apply? Does it work if you reset the top-left screen corner to do nothing, and then make it do "Present Windows - All Desktops"? Tried all corners and edges on neon unstable, none works. (In reply to Nate Graham from comment #6) > Ok cool. > > I had an issue early in the review process with the default top-left screen > corner not working, but that is working for me now. Does it work if you > re-assign it to any other screen corner? no. > Does it wok if you click the "Defaults" button and then Apply? no. > Does it work if you reset the top-left screen corner to do nothing, and then > make it do "Present Windows - All Desktops"? no. That's odd. Do any screen corners work for the Present Windows effect? Is that broken too? Or is only the Desktop Grid broken in screen corner activation? When you set one of these effects in the KCM and move your cursor into the appropriate corner, do you see the glow effect, or not? Tested all actions with upper left corner. Desktop Grid always fails and 'Toggle window switching' fails with the steps below. When the chosen action fails, I don't see the glow effect. To reproduce the bug with 'Toggle window switching' action: 1. choose 'Toggle alternative window switching' action and apply 2. now choose 'Toggle window switching' action and apply 3. try to trigger the chosen action I can reproduce the Desktop Grid effect not working from any screen corners or edges. I thought we were talking about Present Windows earlier, sorry. A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/2464 Git commit 486d229caefa83c90198204ae62b44149a89132d by Nate Graham, on behalf of Vlad Zahorodnii. Committed on 25/05/2022 at 18:11. Pushed by ngraham into branch 'master'. effects/desktopgrid: Fix config group name The config group is formed as "Effect-" + plugin id, i.e. "desktopgrid" Config migration from DesktopGrid to desktopgrid is already handled by a kconf_update script. Part-of: <https://invent.kde.org/plasma/kwin/-/merge_requests/2464> M +1 -1 src/effects/desktopgrid/desktopgridconfig.kcfg https://invent.kde.org/plasma/kwin/commit/486d229caefa83c90198204ae62b44149a89132d Git commit 4c4cb446788269df0b8f1fcba15431d4ca33a146 by Nate Graham, on behalf of Vlad Zahorodnii. Committed on 25/05/2022 at 18:29. Pushed by ngraham into branch 'Plasma/5.25'. effects/desktopgrid: Fix config group name The config group is formed as "Effect-" + plugin id, i.e. "desktopgrid" Config migration from DesktopGrid to desktopgrid is already handled by a kconf_update script. Part-of: <https://invent.kde.org/plasma/kwin/-/merge_requests/2464> (cherry picked from commit 486d229caefa83c90198204ae62b44149a89132d) M +1 -1 src/effects/desktopgrid/desktopgridconfig.kcfg https://invent.kde.org/plasma/kwin/commit/4c4cb446788269df0b8f1fcba15431d4ca33a146 |