[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: [PATCH] Don't show tooltips after a click
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2008-07-25 16:50:55
Message-ID: 200807251050.56117.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Friday 25 July 2008, Jason Stubbs wrote:
> On Saturday 26 July 2008 00:22:08 JST, Jason Stubbs wrote:
> > On Friday 25 July 2008 23:53:29 JST, Alex Merry wrote:
> > > On Friday 25 July 2008 15:36:27 Jason Stubbs wrote:
> > > > Hey,
> > > >
> > > > Just a quickie so going direct to the ML.
> > > >
> > > > Basically, having a tooltip pop up after moving the mouse into the
> > > > tasks and clicking a task to switch to it is really bugging me. This
> > > > patch fixes it (and cleans up a little whitespace too). Okay to
> > > > commit?
> > >
> > > Please do!  It's driving me bats.
> >
> > Done. For some reason, it doesn't fix the Icon applet. Nothing looks
> > strange code-wise though... Will have a closer look.
>
> The icon widget (a child of the icon applet) handles clicks, moves, etc so
> the event never reaches the icon applet and is thus not filterable by the
> tooltip manager. This patch changes the tooltip registration to the icon
> widget.

sounds good.

> Unless! The tooltip manager should be smart enough to filter child widgets
> (and watch for new children) as well? I personally think it's more trouble
> than it's worth, but it'd be the better way to go if it is possible without
> much effort.

no, this will cause other problems (e.g. if ther are different tooltips on 
different children)

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech


["signature.asc" (application/pgp-signature)]

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic