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

List:       kde-pim
Subject:    Re: [Kde-pim] Kaddressbook: featurebarwidget
From:       Tobias Koenig <tokoe () kde ! org>
Date:       2002-11-30 12:29:15
[Download RAW message or body]

On Sat, Nov 30, 2002 at 02:46:26AM +0100, Anders Lund wrote:
> On Friday 29 November 2002 22:47, Tobias Koenig wrote:
Hi Anders,

> Ty Tobias :)
> 
> I have comitted the ViewManager *viewManager() method.
Fine. I would like to add a method ViewManager::addressBook(), so we can
omit the first parameter in FeatureBarWidget and return
  mViewManager->addressBook()
in FeatureBarWidget::addressBook().
So we can make the viewmanager to something like KMKernel in KMail, a
central component that can provide all necessary stuff, that plugins would
need to fulfil their task.

> For the others, I'm not sure it is really nessecary, having this code in 
> show() and hide() is OK as long as those gets called [by the view manager] 
> when the display changes.
> 
> I would like to change ViewManager::showFeatures( id ) though, since it 
> hide()s all feature bars. Why not just 
> 
>     if ( mCurrentFeatureBarWidget )
>       mCurrentFeatureBarWidget->hide();
Well... why not?
Please commit :)

It was late at night when I wrote this code, so maybe I didn't use the
best concept...

Ciao,
Tobias
-- 
In a world without walls and fences who
needs Windows and Gates???
_______________________________________________
kde-pim mailing list
kde-pim@mail.kde.org
http://mail.kde.org/mailman/listinfo/kde-pim
kde-pim home page at http://pim.kde.org/
[prev in list] [next in list] [prev in thread] [next in thread]