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

List:       kde-panel-devel
Subject:    Re: How to check if tooltip is visible (digital clock)?
From:       "Dan Meltzer" <parallelgrapefruit () gmail ! com>
Date:       2008-03-30 21:38:58
Message-ID: 46059ce10803301438r128d7de5taa65e727581ea682 () mail ! gmail ! com
[Download RAW message or body]

On 3/30/08, Rafał Miłecki <zajec5@gmail.com> wrote:
> 2008/3/30, Aaron J. Seigo <aseigo@kde.org>:
> 
> > On Sunday 30 March 2008, Rafał Miłecki wrote:
> > 
> > > I'm surely not experienced enoght to decide which solution is better
> > > (new method, or boolean argument to already existing updateToolTip) \
> > > so
> > 
> > 
> > would you like to try doing up a patch with updateToolTip(bool)? no one \
> > else is using updateToolTip yet, so it shouldn't be any harm.
> 
> 
> Sure, I'll try to figure out how tooptips work and how to add this.
> 
> 
> 
> 2008/3/31, Dmitry Suzdalev <dimsuzkde@gmail.com>:
> 
> > 
> > 
> > On Sunday 30 March 2008 21:57:08 Aaron J. Seigo wrote:
> 
> > > would you like to try doing up a patch with updateToolTip(bool)? no \
> > > one
> > 
> > > else is using updateToolTip yet, so it shouldn't be any harm.
> > 
> > /me has "no booleans in public api" bell ringing :)
> > 
> > Shouldn't that be some kind of enum instead of bool?
> 
> 
> Again, I'm not KDE guru but after checking for example Plasma::Applet
> I can see about 6 bool arguments in public functions:
> http://api.kde.org/4.0-api/kdebase-workspace-apidocs/libs/plasma/html/classPlasma_1_1Applet.html
>  Of course it can also mean that Plasma::Applet should be fixed to use
> enum ;) Just wanted to say what I noticed. If someone asked me about
> this enum<->bool I'd say it sounds weird to use enum for simple
> true/false.

Pssh, everyone knows the following is a better solution than a simple
true/false:

enum truthity {
reallyTrue,
reallyFalse,
FileNotFound
};
> 
> 
> --
> Rafał Miłecki
> _______________________________________________
> Panel-devel mailing list
> Panel-devel@kde.org
> https://mail.kde.org/mailman/listinfo/panel-devel
> 
_______________________________________________
Panel-devel mailing list
Panel-devel@kde.org
https://mail.kde.org/mailman/listinfo/panel-devel


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