Bug 93832

Summary:	kontacts splash screen covers the kwallet password dialog
Product:	[Applications] kontact	Reporter:	Ferdinand Gassauer <gassauer>
Component:	general	Assignee:	kdepim bugs <kdepim-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	l.lunak
Priority:	NOR
Version:	0.1
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Ferdinand Gassauer 2004-11-24 08:29:52 UTC

Version:           0.1 (using KDE 3.3.89 (CVS >= 20041104), compiled sources)
Compiler:          gcc version 3.3.4 (pre 3.3.5 20040809)
OS:                Linux (i686) release 2.6.8-24.3-smp

Hi!
may be this bugreport should go to kwallet.
IMHO the splash screen should not cover the password dialog (or any other dialog box)
cu
ferdinand

Comment 1 Ferdinand Gassauer 2004-11-24 08:30:28 UTC

btw the password dialog pops up, because of kmail

Comment 2 Lubos Lunak 2004-11-24 15:33:49 UTC

Probably rather kwin problem.

Comment 3 Lubos Lunak 2004-11-24 15:51:16 UTC

CVS commit by lunakl: 

Make dialogs an exception from keeping splashscreens on top of every other
window from the app (#93832). IMHO the real problem is in apps showing
modal dialogs while they still have a splascreen visible, but I've seen
this already enough times.
CCBUG: 93832


  M +3 -0      layers.cpp   2.34


--- kdebase/kwin/layers.cpp  #2.33:2.34
@@ -608,4 +608,7 @@ bool Workspace::keepTransientAbove( cons
     if( mainwindow->isTopMenu() && transient->groupTransient())
         return false;
+    // #93832 - don't keep splashscreens above dialogs
+    if( transient->isSplash() && mainwindow->isDialog())
+        return false;
     // This is rather a hack for #76026. Don't keep non-modal dialogs above
     // the mainwindow, but only if they're group transient (since only such dialogs

Comment 4 Lubos Lunak 2004-11-24 15:53:15 UTC

Also backported for 3.3.2.

Comment 5 Tobias Koenig 2004-11-26 00:24:24 UTC

Fixed by Lubos' patch