Session management hickup

[Lubos Lunak]

On Sunday 13 of June 2004 19:43, Rob Kaper wrote:
> From  bool saveState( QSessionManager& ) in KMWSessionManaged, around
> kmainwindow.cpp  line 99:
>
>         for (it.toFirst(); it.current(); ++it){
>             if( it.current()->isVisible()) {
>                 n++;
>                 it.current()->savePropertiesInternal(config, n);
>             }
>         }
>
> Why is the isVisible here?

 cvs annotate is your friend : 
http://webcvs.kde.org/cgi-bin/cvsweb.cgi/kdelibs/kdeui/kmainwindow.cpp#rev1.105

 I think it could be reverted and that particular problem fixed directly in 
Konqueror.

> Is it for Konqueror's preloaded windows?

 No.

> It  
> seems to break session management for apps that allow the mainwindow to be
> hidden for a system tray icon (LightHawk, and I recall similar issues with
> Kopete).
>
> (regression would be since 3.2)
>
> Rob

-- 
Lubos Lunak
KDE developer
---------------------------------------------------------------------
SuSE CR, s.r.o.  e-mail: l.lunak at suse.cz , l.lunak at kde.org
Drahobejlova 27  tel: +420 2 9654 2373
190 00 Praha 9   fax: +420 2 9654 2374
Czech Republic   http://www.suse.cz/