| Summary: | window border "stays on top" after using present windows | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | Franz Trischberger <franz.trischberger> |
| Component: | general | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | Flags: | thomas.luebking:
ReviewRequest+
|
| Priority: | NOR | ||
| Version: | 4.10.0 | ||
| Target Milestone: | 4.10.1 | ||
| Platform: | Other | ||
| OS: | Linux | ||
| URL: | http://git.reviewboard.kde.org/r/108867/ | ||
| Latest Commit: | http://commits.kde.org/kde-workspace/42e547571e963f4c63b4ae77b00cbf2d6652b5ad | Version Fixed In: | 4.10.1 |
| Sentry Crash Report: | |||
| Attachments: | s/panelproxy heuristic/screenedge restack | ||
|
Description
Franz Trischberger
2013-02-07 20:34:46 UTC
Not saying it's related, but stay tuned on bug #313145 Thanks. I will go through the other bug report tomorrow. It seemed as if just bespin deco was affected, but after setting oxygen border to "none" this happens with oxygen, too. Yes, it does seem we uncovered another bug from the "ancient ages of doom" Bespin and oxygen w/o borders support the input window (ie. resizing the window w/o visual borders) what apparently exposes this bug on 64bit architectures. Do you compile -O3? I compile with -O2. So does Alex, but it actually doesn't matter (could as well be something in QVector or whatnot) The other bug has a patch attached. Feel free to try. As I'm running 4.10.0 I had to modify the patch. It did not fix this issue. > As I'm running 4.10.0 I had to modify the patch.
> It did not fix this issue.
then I think it's a different issue
Is the window "near" the edge? In screenedge.cpp shortcut void ScreenEdge::raisePanelProxies() (return immediately) - does the issue remain? Created attachment 77024 [details]
s/panelproxy heuristic/screenedge restack
Reporter says: "yes, shortcut seems to do", thus i suspect a false positive panelproxy hit on the input window(s)
@Franz
Please test the attached patch to substitute raising proxies by lowering screenedges.
Yes, the attached patch fixes the issue. Thx :) Git commit af37273c79d2da157a6821788676aad1ab522789 by Thomas Lübking. Committed on 08/02/2013 at 20:59. Pushed by luebking into branch 'KDE/4.10'. do not try to raise possible panel proxies but drop screenedges below the supportWindow instead that's why it exists, that's deterministic, that's faster FIXED-IN: 4.10.1 REVIEW: 108867 M +1 -1 kwin/effects.cpp M +23 -0 kwin/layers.cpp M +0 -49 kwin/screenedge.cpp M +0 -5 kwin/screenedge.h M +1 -0 kwin/workspace.h http://commits.kde.org/kde-workspace/af37273c79d2da157a6821788676aad1ab522789 Git commit 42e547571e963f4c63b4ae77b00cbf2d6652b5ad by Thomas Lübking. Committed on 08/02/2013 at 20:59. Pushed by luebking into branch 'master'. do not try to raise possible panel proxies but drop screenedges below the supportWindow instead that's why it exists, that's deterministic, that's faster includes adaption to new screenedge and xcb invocation (compared to 4.10) FIXED-IN: 4.10.1 REVIEW: 108867 M +1 -1 kwin/effects.cpp M +14 -0 kwin/layers.cpp M +0 -64 kwin/screenedge.cpp M +0 -5 kwin/screenedge.h M +4 -0 kwin/workspace.h http://commits.kde.org/kde-workspace/42e547571e963f4c63b4ae77b00cbf2d6652b5ad |