Bug 482641 - Cannot trigger desktop grid using Screen Edges
Summary: Cannot trigger desktop grid using Screen Edges
Status: RESOLVED DUPLICATE of bug 478137
Alias: None
Product: kwin
Classification: Plasma
Component: effects-desktop-grid (show other bugs)
Version: 6.0.1
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords: qt6
Depends on:
Blocks:
 
Reported: 2024-03-07 04:53 UTC by Rohitt Vashishtha
Modified: 2024-03-07 13:35 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
screenshot showing that the "Grid" enry is missing (90.30 KB, text/plain)
2024-03-07 07:23 UTC, Rohitt Vashishtha
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Rohitt Vashishtha 2024-03-07 04:53:03 UTC
SUMMARY


STEPS TO REPRODUCE
1. Upgrade to Plasma 6 and observe that any previous binding in Screen Edges to show the desktop grid disappears, and there's no such entry in the list anymore

OBSERVED RESULT


EXPECTED RESULT

Have Desktop Grid in the Screen Edges shortcuts list similar to how Overview exists.

SOFTWARE/OS VERSIONS

Operating System: Arch Linux 
KDE Plasma Version: 6.0.1
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.2
Kernel Version: 6.6.20-1-lts (64-bit)
Graphics Platform: Wayland
Graphics Processor: Mesa Intel® UHD Graphics 620

ADDITIONAL INFORMATION

I know that the overview effect replaces the Desktop Grid effect, so the existing binding getting unset wasn't surprising at all, but being unable to toggle the grid while being able to toggle the overview with screen-edges feels like a minor oversight and probably easy to add hopefully, since toggling with Meta+G works perfectly.
Comment 1 Rohitt Vashishtha 2024-03-07 07:23:21 UTC
Created attachment 166548 [details]
screenshot showing that the "Grid" enry is missing

I checked the code and it seems like the linked commit in master should be fixing this. But I don't understand how come this isn't in the 6.0.1 tree/release even though its 10 days old? Just trying to understand the release process a bit. Would it later be cherry-picked to the Plasma/6.0 branch or was it never intended for Plasma 6?

commit that fixes: https://invent.kde.org/plasma/kwin/-/commit/4d6f6223bcdbb0e5fbe65cff47c72d444b532372
Comment 2 Vlad Zahorodnii 2024-03-07 13:35:54 UTC
> But I don't understand how come this isn't in the 6.0.1

Because it involves changing strings so 6.1

*** This bug has been marked as a duplicate of bug 478137 ***