Bug 193155

Summary: changing the volume with the mouse will doesn't show the tooltip
Product: [Applications] kmix Reporter: Giulio Camuffo <giuliocamuffo>
Component: generalAssignee: Christian Esken <esken>
Status: RESOLVED FIXED    
Severity: normal CC: emdeck, gorb82
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:

Description Giulio Camuffo 2009-05-18 22:00:47 UTC
Version:            (using KDE 4.2.85)
OS:                Linux
Installed from:    Unlisted Binary Package

The new systray tooltip is really cool, but it has a little problem: when I move the mouse wheel over the kmix icon it won't show up the tooltip with the percentage of the volume until i stop using the wheel and wait the prefixed time before the showing up.
Comment 1 Christian Esken 2009-07-05 18:12:38 UTC
This was also so before in KDE4.2, KDE3.5, ..., wouldn't you agree?
Comment 2 Giulio Camuffo 2009-07-05 21:14:51 UTC
It seems to me that, at least in 4.2, the tooltip appeared immediately.
Anyway I think that it would be really useful if it acted that way.
Comment 3 Ilya 2009-08-27 10:32:27 UTC
I confirm that in 4.2 to work properly. now have to wait some time before the advent of tooltip
Comment 4 Vitaliy 2009-08-27 12:54:01 UTC
I confirm this bug.
Comment 5 MichaƂ D. (Emdek) 2009-09-22 15:58:47 UTC
This regression was introduced when porting to new specification:

http://reviewboard.kde.org/r/621/

This code was probably used to trigger tool tip and it has no equivalent now (I couldn't find any in new API):

366	    // Simulate a mouse move to make Qt show the tooltip again		
367	    // (Qt removes it on a MouseWheel event)		
368	    QApplication::postEvent( this, new QHelpEvent ( QEvent::ToolTip, e->pos(), e->globalPos() ) );
Comment 6 Marco Martin 2009-09-22 20:31:40 UTC
SVN commit 1026853 by mart:

don't reset the timer for 0 distance hover move events
BUG:193155


 M  +7 -2      tooltipmanager.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1026853
Comment 7 Christian Esken 2009-09-24 00:32:18 UTC
Thanks for that patch, Marco.
I don't think I would have ever found that.