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

List:       kfm-devel
Subject:    Re: PATCH adds PluginInterface to KonqPopupMenu
From:       Holger Freyther <freyther () gmx ! net>
Date:       2001-10-30 13:17:15
[Download RAW message or body]

On Tuesday 30 October 2001 14:12, David Faure wrote:
> On Mardi 30 Octobre 2001 13:41, Simon Hausmann wrote:
> > Hm, ok, we already have popup( const QPoint & ) , so I guess another
> > stub doesn't hurt. Hmmm, I agree on an insert( KActionMenu * ) stub
> > (David, what do you think?)
>
> KActionMenu already has insert(KAction *). And KActionMenu is an action
> so in terms of API it doesn't make much sense to have both.
>
> Doesn't insert(KAction*) work for kactionmenus? It calls plug, which
> inserts the whole popupmenu, for action menus, doesn't it ?
>
> -> I don't see the need for this, it seems to me that insert(KAction *)
> does it all already.
>
> David.
read some posts before. Yes it should work with a down_cast but it would work 
Simons way but I thought it was cleaner this was not eeding to do this cast 
in an application
regards Holger

-- 
Kwebsuite http://kwebsuite.sf.net Holger Freyther alias zecke(123)
freyther@yahoo.com
freyther@gmx.net
spamme@keul-dir-ein.de for spmmers

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