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

List:       kde-pim
Subject:    Re: [Kde-pim] Kaddressbook column width bug
From:       Tobias Koenig <tokoe () kde ! org>
Date:       2002-09-07 8:46:04
[Download RAW message or body]

On Fri, Sep 06, 2002 at 12:03:18AM +0200, Tobias Koenig wrote:
> On Thu, Sep 05, 2002 at 11:03:33PM +0200, Cornelius Schumacher wrote:
> > On Thursday 05 September 2002 18:33, Tobias Koenig wrote:
> > > On Thu, Sep 05, 2002 at 10:33:05PM +1000, David Findlay wrote:
> > > Hi David,
> > >
> > > > I note that this bug still exists in 3.0.3. Is it now fixed in CVS
> > >
> > > Yes
> > 
> > Is it possible to also fix it in the stable branch?
> I guess a KListView::saveLayout() and KListView::restoreLayout() at the
> right places should do the trick.
> Whats about the attached patch?
Could somebody with a working KDE 3.0.3 please test the patch and commit
it to CVS BRANCH when working?

> Index: contactlistview.cpp
> ===================================================================
> RCS file: /home/kde/kdebase/kaddressbook/Attic/contactlistview.cpp,v
> retrieving revision 1.10
> diff -u -b -p -r1.10 contactlistview.cpp
> --- contactlistview.cpp	2002/02/13 10:49:27	1.10
> +++ contactlistview.cpp	2002/07/23 20:58:58
> @@ -299,6 +299,7 @@ void ContactListView::saveConfig()
>    KConfig *config = kapp->config();
>  
>    config->setGroup("ListView");
> +  saveLayout( config, config->group() );
>    config->writeEntry( "sortColumn", header()->mapToActual( column ));
>    config->writeEntry( "sortDirection", ascending );
>  
> @@ -317,6 +318,7 @@ void ContactListView::readConfig()
>    KConfig *config = kapp->config();
>  
>    config->setGroup("ListView");
> +  restoreLayout(config, config->group());
>    column = config->readNumEntry( "sortColumn", 0 );
>    ascending = config->readBoolEntry( "sortDirection", true );
>  

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]