| Summary: | Inactive inner window handling - impossible to click if click is not configured to pass | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | pawel.veselov |
| Component: | general | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | nate |
| Priority: | NOR | Keywords: | regression |
| Version First Reported In: | 6.4.0 | ||
| Target Milestone: | --- | ||
| Platform: | Fedora RPMs | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/kwin/-/commit/becb3abe9a1c85a9ec533f202ae3c2b2d06ed2d1 | Version Fixed/Implemented In: | 6.4.1 |
| Sentry Crash Report: | |||
|
Description
pawel.veselov
2025-06-22 22:14:50 UTC
Regression after https://invent.kde.org/plasma/kwin/-/commit/87bb2447e5328a5e6f4d066fcfea5ff38a02139d A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/7800 Git commit e8dad997fae98e3c0c40653667dd22a002747556 by Vlad Zahorodnii. Committed on 23/06/2025 at 12:22. Pushed by vladz into branch 'master'. Fix Window::mousePressCommandConsumesEvent() for "activate" action Amends 87bb2447e5328a5e6f4d066fcfea5ff38a02139d. According to the boolean algebra rules, if the `a || b` expression is negated, the result will be `!a && !b`. In the prior version, the MouseActivate option used to correspond to `isActive() || !rules()->checkAcceptFocus(acceptsFocus())` so following the boolean algebra rules, the `||` operator should be changed to `&&`. M +1 -1 src/window.cpp https://invent.kde.org/plasma/kwin/-/commit/e8dad997fae98e3c0c40653667dd22a002747556 Git commit becb3abe9a1c85a9ec533f202ae3c2b2d06ed2d1 by Vlad Zahorodnii. Committed on 23/06/2025 at 17:42. Pushed by vladz into branch 'Plasma/6.4'. Fix Window::mousePressCommandConsumesEvent() for "activate" action Amends 87bb2447e5328a5e6f4d066fcfea5ff38a02139d. According to the boolean algebra rules, if the `a || b` expression is negated, the result will be `!a && !b`. In the prior version, the MouseActivate option used to correspond to `isActive() || !rules()->checkAcceptFocus(acceptsFocus())` so following the boolean algebra rules, the `||` operator should be changed to `&&`. (cherry picked from commit e8dad997fae98e3c0c40653667dd22a002747556) Co-authored-by: Vlad Zahorodnii <vlad.zahorodnii@kde.org> M +1 -1 src/window.cpp https://invent.kde.org/plasma/kwin/-/commit/becb3abe9a1c85a9ec533f202ae3c2b2d06ed2d1 |