Summary: | Screen edge effect does not correspond with actual length of edge that can be triggered | ||
---|---|---|---|
Product: | [Plasma] kwin | Reporter: | fanzhuyifan |
Component: | general | Assignee: | KWin default assignee <kwin-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | minor | CC: | nate |
Priority: | NOR | ||
Version: | git master | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
See Also: | https://bugs.kde.org/show_bug.cgi?id=464856 | ||
Latest Commit: | https://invent.kde.org/plasma/kwin/-/commit/4727a7d956318086c4456d8ade3a22803eb8ff47 | Version Fixed In: | 6.0 |
Attachments: | video |
Description
fanzhuyifan
2024-02-13 01:43:00 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/5179 Git commit 2f4db693e3c826be7ac76629364630276e65bc25 by Yifan Zhu. Committed on 13/02/2024 at 15:30. Pushed by fanzhuyifan into branch 'master'. screenedge: don't reduce approachGeometry Reducing approachGeometry is not needed. During edge creation, createHorizontalEdge and createVerticalEdge already substract the needed cornerOffset, and edges reserved by client windows shouldn't be reduced in geometry. In addition, during display, ScreenEdgeEffect confines the painted area to approachGeometry. So approachGeometry shouldn't be reduced in advance. M +0 -8 src/screenedge.cpp https://invent.kde.org/plasma/kwin/-/commit/2f4db693e3c826be7ac76629364630276e65bc25 Git commit 4727a7d956318086c4456d8ade3a22803eb8ff47 by Yifan Zhu. Committed on 13/02/2024 at 15:41. Pushed by fanzhuyifan into branch 'Plasma/6.0'. screenedge: don't reduce approachGeometry Reducing approachGeometry is not needed. During edge creation, createHorizontalEdge and createVerticalEdge already substract the needed cornerOffset, and edges reserved by client windows shouldn't be reduced in geometry. In addition, during display, ScreenEdgeEffect confines the painted area to approachGeometry. So approachGeometry shouldn't be reduced in advance. (cherry picked from commit 2f4db693e3c826be7ac76629364630276e65bc25) M +0 -8 src/screenedge.cpp https://invent.kde.org/plasma/kwin/-/commit/4727a7d956318086c4456d8ade3a22803eb8ff47 Sort of mutually exclusive with Bug 464856; maybe we should mark that as INTENTIONAL? Or change this around to implement that, rather than this? (In reply to Nate Graham from comment #4) > Sort of mutually exclusive with Bug 464856; maybe we should mark that as > INTENTIONAL? Or change this around to implement that, rather than this? I don't think it is mutually exclusive -- we can just make the panel reserve the whole edge and we could solve both issues. The commit fixing this was only about the visual effect code, and making it consistent with what was requested. I see, cool. (In reply to Nate Graham from comment #4) > Sort of mutually exclusive with Bug 464856; maybe we should mark that as > INTENTIONAL? Or change this around to implement that, rather than this? Oh do you mean in that case we would want the edge trigger effect to only highlight the panel, but make the edge trigger everywhere? That would require deeper changes to the current code I think. My thinking would be that the entire screen edge would both highlight visually and also act as a triggering area. So the visual effect would match the triggering area, not the panel width. |