| Summary: | Padding not remembered properly per display, but UI implies it is | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | m1vri31c |
| Component: | Custom Tiling | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | 4wy78uwh, kde, nate, notmart, tobias.fella |
| Priority: | NOR | Keywords: | multiscreen, usability |
| Version First Reported In: | 6.0.5 | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/kwin/-/commit/d3c093eabce38bd85638fd599f3a6fafac6074fe | Version Fixed/Implemented In: | 6.6.0 |
| Sentry Crash Report: | |||
| Attachments: | A Screenshot Of The Potentially-Relevant Preference | ||
|
Description
m1vri31c
2024-06-07 05:33:45 UTC
Update: It is not always the padding setting that was last set that is maintained. It's not stored on a per-screen basis. The code is working as expected, but the UI is misleading showing on every screen. We can change that to only be on the primary screen. (In reply to David Edmundson from comment #2) > It's not stored on a per-screen basis. > The code is working as expected, but the UI is misleading showing on every > screen. > > We can change that to only be on the primary screen. It would be great if it could store it per-screen in the future, since larger monitors often feel better with more padding and vice versa. (In reply to David Edmundson from comment #2) > It's not stored on a per-screen basis. > The code is working as expected, but the UI is misleading showing on every > screen. It's not just the UI being misleading, at runtime the padding is applied per-screen. > We can change that to only be on the primary screen. I'd propose properly allow setting and saving it per-screen, the change required for that seems minimal. A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/8732 Git commit 348d0a0381f7bc7eb04c8fc5db5dc725a1a531a0 by Vlad Zahorodnii, on behalf of Tobias Fella. Committed on 04/02/2026 at 10:11. Pushed by vladz into branch 'master'. Save tiling padding per screen The current UI already sets it per-screen at runtime, but only saves it globally. When restarting, the padding then appears different than the user expects. Instead, save the padding per screen. M +4 -2 src/tiles/tilemanager.cpp https://invent.kde.org/plasma/kwin/-/commit/348d0a0381f7bc7eb04c8fc5db5dc725a1a531a0 Git commit d3c093eabce38bd85638fd599f3a6fafac6074fe by Vlad Zahorodnii, on behalf of Tobias Fella. Committed on 04/02/2026 at 12:02. Pushed by vladz into branch 'Plasma/6.6'. Save tiling padding per screen The current UI already sets it per-screen at runtime, but only saves it globally. When restarting, the padding then appears different than the user expects. Instead, save the padding per screen. (cherry picked from commit 348d0a0381f7bc7eb04c8fc5db5dc725a1a531a0) M +4 -2 src/tiles/tilemanager.cpp https://invent.kde.org/plasma/kwin/-/commit/d3c093eabce38bd85638fd599f3a6fafac6074fe Created attachment 189415 [details] A Screenshot Of The Potentially-Relevant Preference If we refer to the Custom Tiling padding, I would prefer the padding preference to be applied to all displays, because: - I frequently add and remove displays. - I utilise the same padding on each: - …to work around https://discuss.kde.org/t/31979/6?u=rokejulianlockhart. - …because it *was* useful for screenshots, before https://discuss.kde.org/t/44175?u=rokejulianlockhart (so it likely shall be again, when/if resolved). - …because 3 pixels prevents the bottom-most window border rendering below the display. |