From kde-core-devel Mon Jun 14 10:14:47 2004 From: Lubos Lunak Date: Mon, 14 Jun 2004 10:14:47 +0000 To: kde-core-devel Subject: Re: Session management hickup Message-Id: <200406141214.48036.l.lunak () suse ! cz> X-MARC-Message: https://marc.info/?l=kde-core-devel&m=108725860103261 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@suse.cz , l.lunak@kde.org Drahobejlova 27 tel: +420 2 9654 2373 190 00 Praha 9 fax: +420 2 9654 2374 Czech Republic http://www.suse.cz/