Summary: | Horizontal and Vertical maximized states are reversed in signal parameters | ||
---|---|---|---|
Product: | [Plasma] kwin | Reporter: | Chris Holland <zrenfire> |
Component: | scripting | Assignee: | kde.org |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | kde.org, nate |
Priority: | NOR | ||
Version: | 5.9.5 | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/plasma/kwin/commit/f6f4a296f26215d36eb1f12c66d3b485f5b7e825 | Version Fixed In: | |
Attachments: | kwin console log output |
Description
Chris Holland
2017-06-04 19:36:21 UTC
most of the code has been refactored, but I found one location in abstract_client.cpp where the arguments were swapped. I created https://invent.kde.org/plasma/kwin/-/merge_requests/1610 to fix this. Here's the latest (permanent) links that take into account the refactor of `src/` and the removal of `geometry.cpp` for posterity. * https://github.com/KDE/kde-workspace/blame/257b997132217fd73a1b3be14db49735b64235e7/kwin/netinfo.cpp#L259 * https://github.com/KDE/kwin/blame/84a5c7b314965853fb21cda8128809ebf4af795a/src/abstract_client.h#L515 * https://github.com/KDE/kwin/blame/84a5c7b314965853fb21cda8128809ebf4af795a/src/abstract_client.cpp#L917 * https://github.com/KDE/kwin/blame/84a5c7b314965853fb21cda8128809ebf4af795a/src/abstract_client.h#L939 * https://github.com/KDE/kwin/blame/84a5c7b314965853fb21cda8128809ebf4af795a/src/scripting/workspace_wrapper.cpp#L304 * https://github.com/KDE/kwin/blame/84a5c7b314965853fb21cda8128809ebf4af795a/src/scripting/workspace_wrapper.h#L92 Thank you for your feedback. I'll have a look the the other files as well and update my merge request to fix this. Git commit f6f4a296f26215d36eb1f12c66d3b485f5b7e825 by Vlad Zahorodnii, on behalf of Guenther Grau. Committed on 19/05/2022 at 06:33. Pushed by vladz into branch 'master'. Fix argument order for clientMaximizedStateChanged Arguments for clientMaximizedStateChanged were incorrect in window.cpp. They are correct in xdgshellclient.cpp and window.h M +1 -1 src/window.cpp https://invent.kde.org/plasma/kwin/commit/f6f4a296f26215d36eb1f12c66d3b485f5b7e825 |