Session management hickup

[Rob Kaper]

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? Is it for Konqueror's preloaded windows? 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
-- 
Rob Kaper     | "Whoever wants to know the heart and mind of America had
cap at capsi.com | better learn baseball" -- Jacques Barzun
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20040613/0ca5f3cf/attachment.sig>