Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 364555 - kde-base/plasma-workspace-4.6.2 panel misplacement, regression from 4.6.1, libkdeinit4_plasma-desktop.so
Summary: kde-base/plasma-workspace-4.6.2 panel misplacement, regression from 4.6.1, li...
Status: RESOLVED NEEDINFO
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] KDE (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo KDE team
URL: https://bugs.kde.org/show_bug.cgi?id=...
Whiteboard: tracking upstream
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-23 07:32 UTC by Duncan
Modified: 2011-11-13 00:09 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
emerge --info plasma-workspace (plasma-workspace.emerge.info,6.49 KB, text/plain)
2011-04-23 07:32 UTC, Duncan
Details
Patch reverting upstream commit 181c58c354f15fb1bda91ea6582f2baf3975c7f2 (plasma-wkspc-pnl-resize-revert.patch,2.85 KB, patch)
2011-05-07 10:00 UTC, Duncan
Details | Diff
Corrected: Patch reverting upstream commit b239193e10195b13943686f39a186cd29fbf7e24 (rehide-revert.patch,13.38 KB, patch)
2011-05-07 13:01 UTC, Duncan
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Duncan 2011-04-23 07:32:58 UTC
Created attachment 270943 [details]
emerge --info plasma-workspace

The URL has the kde-upstream bug, just filed, and more detail, including correct and misplaced panel desktop images as attachments.

The gist is that in a regression from 4.6.1, plasma-workspace 4.6.2 misplaces my top panel from the top monitor in a dual-monitor/stacked-layout desktop, to a few pixels /below/ the top of the /bottom/ monitor.

By replacing individual files from 4.6.2 with those from the 4.6.1 binpkg, I was able to isolate the individual problem file within the package, libkdeinit4_plasma-desktop.so.

This could be a blocker for kde 4.6.2 stabilization, particularly as it's a regression from 4.6.1.  But being nailed down to the specific library file and a single micro-release update, finding and reverting or patching the bad commit shouldn't be difficult.  (If the browse sources link was actually somewhere I could find on the kde site, I might have already isolated the individual commit.  Looking forward to the completion of the git transition such that I could potentially bisect it, as well.)

Full emerge --info plasma-workspace attached.  Summary: ~amd64/no-multilib, portage-2.2.0_alpha30, kde and x11 overlays active, USE flags include semantic-desktop and xinerama.  Perhaps worth noting, xorg-server-1.10.1, radeon-native classic driver in kms mode, kernel 2.6.38.
Comment 1 Andreas K. Hüttel archtester gentoo-dev 2011-04-24 22:22:54 UTC
quickgit.kde.org is your friend :)

maybe this here might have to do with your problem?

http://bugs.kde.org/show_bug.cgi?id=220088
Comment 2 Duncan 2011-05-07 01:00:54 UTC
(In reply to comment #1)
> maybe this here might have to do with your problem?
> 
> http://bugs.kde.org/show_bug.cgi?id=220088

AFAICT, unrelated, as that bug is desktop-grid related, while mine is simply the panel appearing in the wrong place (a few px below the top edge, not /at/ the top edge where it'd normally dock) on the wrong monitor (the bottom one of two stacked, instead of the top one), at plasma-desktop startup, no desktop-grid activation of any sort required.

Meanwhile, the regression is confirmed to remain in 4.6.3 as well.

I've been meaning to check quickgit, now that you pointed me at it (thanks), but haven't gotten the properly rounded tuit, yet, unfortunately. =:^(
Comment 3 Duncan 2011-05-07 10:00:47 UTC
Created attachment 272361 [details, diff]
Patch reverting upstream commit 181c58c354f15fb1bda91ea6582f2baf3975c7f2

The upstream bug is now bisected.  Here's the patch reverting it.

Tested by dumping it in /etc/portage/patches/kde-base/plasma-workspace and rebuilding the package, verifying that the patch was applied.  The patch was created against 4.6.3 but is tested to apply and build against both 4.6.3 and 4.6.2-r1.   Both resulting packages run fine when merged against a 4.6.3 kde.
Comment 4 Duncan 2011-05-07 13:01:24 UTC
Created attachment 272375 [details, diff]
Corrected: Patch reverting upstream commit b239193e10195b13943686f39a186cd29fbf7e24

Let's actually get the /correct/ patch this time!  (I screwed up testing on the last one and was testing this one the entire time!  GRRR!!)
Comment 5 Andreas K. Hüttel archtester gentoo-dev 2011-05-21 20:33:51 UTC
Patch added in 4.6.2-r2 and 4.6.3-r1. Let's wait for the fallout, after all that commit was intended to fix something...
Comment 6 Duncan 2011-05-23 08:53:17 UTC
(In reply to comment #5)
> Patch added in 4.6.2-r2 and 4.6.3-r1. Let's wait for the fallout, after all
> that commit was intended to fix something...

Confirmed that 4.6.3-r1 works fine, here (after removing the /etc/portage/patches/kde-base/plasma-workspace patch I had applied locally).

Based on the upstream commit comments and an apparently related bug (kde bug #272663, auto-hiding panel sometimes moves to the middle of the screen when being viewed), the patch fixed notification behavior in the case of a hidden panel containing the systray/notifier.

That would have been fine, except that apparently, the new code fails to account for the panel being a dock window, so if other (normal) windows are "in the way", as in upstream bug 272663, or in the case of a dual monitor setup, as in my bug, the new code apparently tries to position the affected panel as it would a normal window.  (Having the panel treated as a normal window also probably explains why it was appearing a few pixels below the top of the bottom monitor in my case, as I have a window rules for a couple apps that positions them that way, that was apparently triggering for the panel window, once it was no longer treated as a dock window, as well.)

That doesn't work so well for a panel/dock-window, so the committed fix for the unhide that includes the code ends up being worse than the problem it was trying to fix.  As such, I doubt the fallout will be /too/ bad.  Certainly not if the alternative triggers auto-hide panels appearing at the wrong place on single-monitor setups as well as my bug, which would seem to be the case if indeed the two upstream bugs are related.

Never-the-less, please cc me on any bugs you see that appear to result from this.

FWIW, that other bug suggests a different workaround as well -- a special window rule forcing no kwin auto-window-placement for the affected panel window.  I hadn't thought of that as I'm not used to thinking in terms of panels being windows to which window rules apply, but it should work.  So if this patch does end up reverted before upstream fixes the issue correctly, that could be a viable workaround.
Comment 7 Andreas K. Hüttel archtester gentoo-dev 2011-05-23 17:58:16 UTC
Is getting reverted right now, the complaints are already getting in. (Considering that panel autohide is probably more frequently used than multiscreen setup...)
Comment 8 Herbert Wantesh 2011-05-29 12:07:52 UTC
there is a workaround as mentioned on bugs.kde.org 
go to settings -> Windows behavior (its Fenterverhalten in german) -> Window Rules -> create a new rule -> get window properties -> select the panel ans set
Window Class: Plasma
Window Role: Dock(panel)
this two options should be set with "get window properties"
Geometry Option: Placement = Force: No Placement

i hope i translated the words in the right way ;)
Comment 9 Andreas K. Hüttel archtester gentoo-dev 2011-07-11 20:48:59 UTC
How is the situation in kde 4.6.4 or 4.6.5?
Comment 10 Andreas K. Hüttel archtester gentoo-dev 2011-08-27 19:13:47 UTC
No response for >6 weeks
Comment 11 Herbert Wantesh 2011-11-13 00:09:23 UTC
it seems i solved the bug at least the panel stays where it should stay for the last few hours...

https://bugs.kde.org/show_bug.cgi?id=277248#c7 describes the solution:


kquitapp plasma-desktop 

mv ~/.kde/share/config/plasma-desktop-appletsrc
~/.kde/share/config/plasma-desktop-appletsrc.backup 
mv ~/.kde/share/config/plasma-desktoprc
~/.kde/share/config/plasma-desktoprc.backup  

plasma-desktop

after this the whole plasma desktop has to be reconfigured ...