Bug 157816 - Hovering any icon on the panel shows the thumbnail of latest shown running app.
Summary: Hovering any icon on the panel shows the thumbnail of latest shown running app.
Status: RESOLVED WORKSFORME
Alias: None
Product: plasma4
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
: 157821 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-02-14 12:05 UTC by Vincenzo Di Massa
Modified: 2008-02-26 14:24 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Vincenzo Di Massa 2008-02-14 12:05:22 UTC
Version:           svn Feb 14 2008 (using Devel)
Installed from:    Compiled sources
OS:                Linux

To reproduce the bug follow theese steps:
1) enable app thumbnails when hovering on the taskbar
2) hover any panel icon (it shows the hint nicely)
3) hover any taskbar app (it shows the thumbnail nicely)
4) hover again any panel icon (it shows the thumbnail from step3)

I'm using an R9600 card with latest (8.01) fglrx on a kubuntu gutsy kernel.

I'm not sure plasma is responsible for this bug (it could be kwin) but my guess is that it is the panel/icon code the one who triggers the thumbnail display.

Vincenzo
Comment 1 Aaron J. Seigo 2008-02-14 17:34:36 UTC
i've actually looked into this, and we're setting the window id correctly when mousing over different taskbar buttons.
Comment 2 Aaron J. Seigo 2008-02-14 17:36:42 UTC
*** Bug 157821 has been marked as a duplicate of this bug. ***
Comment 3 Vincenzo Di Massa 2008-02-17 11:56:28 UTC
I'm compiling just now what (from the commit log) appears to be the fix to this problem. I'll update the report once I have recompiled :-)
Comment 4 Vincenzo Di Massa 2008-02-17 19:19:54 UTC
The commit I mentioned before did not fix this.
I was not able to understand where does KWin write to the ToolTip WindowPreview...
I would have liked to fix it :-)


Comment 5 Lubos Lunak 2008-02-24 13:56:35 UTC
SVN commit 778721 by lunakl:

Backport r776040 (detect property changes, fixes taskbar tooltips).
BUG: 157816



 M  +1 -1      unmanaged.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=778721
Comment 6 Vincenzo Di Massa 2008-02-24 20:03:10 UTC
I'm on r778759 and still suffer from the bug.
The fix fixed the taskbar, but not the icons on the panel.

That's why I'm reopening.
Comment 7 Vincenzo Di Massa 2008-02-24 23:53:08 UTC
I read both kwin and plasma related code. 
Both of them read and set the property correctly... is it an Xorg bug?

Plasma icon enables and then disables the property for the tootip windows... maybe that "disabling" does not do what it is supposed to do?

When kwin reads back the property it is still set.

I ve spent quite some time investigating... but this bug wins ATM.

Comment 8 Sebastian Sauer 2008-02-25 01:49:00 UTC
Hi Vincenzo,

you may try to move the mouse first up (out of the taskbar/panel to the desktop) and then to the other taskbar. So far what I did note is, that the thumbnail doesn't got updated if moving direct from one taskitem to another (aka without moving the mouse before to the desktop out of the panel).

Looks for me somewhat like a dup or at least related to bug #154697 - the mouseleave-event doesn't got always emitted correct as it seems :-/
Comment 9 Vincenzo Di Massa 2008-02-25 10:57:48 UTC
Sebastian, 
what you say in #8 has been fixed with r776040: the problem is that even now that the tooltips (the kwin Unmanaged class) react properly to mouse events, kwin writes a window preview in them.

As I said I read the code, so I know what I'm talking about, but it just looks correct correct even if it isn't :-(

Both lunak and aseigo have reviewed this bug, and three of us agree both kwin and plasma look correct :-O
Comment 10 Vincenzo Di Massa 2008-02-26 14:22:03 UTC
SVN commit 779551 by dimassa:

Backport of r779541 to 4.0 branch

CCBUG: 157816



 M  +2 -2      tooltip.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=779551
Comment 11 Vincenzo Di Massa 2008-02-26 14:24:04 UTC
r779541 did it