Bug 62932

Summary:	gideon splash screen placement
Product:	[Applications] kdevelop	Reporter:	Roger Larsson <roger.larsson>
Component:	general	Assignee:	KDevelop Developers <kdevelop-devel>
Status:	RESOLVED FIXED
Severity:	minor	CC:	l.lunak
Priority:	NOR
Version:	3.0.0a5
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Roger Larsson 2003-08-19 02:23:08 UTC

Version:           3.0.0a5 (using KDE 3.1.9)
Compiler:          gcc version 3.2
OS:          Linux (i686) release 2.4.19-4GB

The splash screen in gideon is not placed in the center of the window but rather on a static offset from upper left corner.

When I first started gideon the window it started with was not full screen - so I could not see the full splash.

Not looking good.

Comment 1 tnagy 2003-09-11 20:54:54 UTC

In addition the splashscreen is blue whereas the gideon icon is green (color
inconsistency)

Comment 2 Ravikiran Rajagopal 2003-09-13 19:11:16 UTC

Subject: [PATCH] Fix for bug 62932

Hello,
  The following patch fixes bug http://bugs.kde.org/show_bug.cgi?id=62932 and 
makes sures that the splash screen stays on top. The splash screen widget 
should have no parent as it should be a toplevel widget; see other splash 
screens in KDE CVS. Also, I did not find a license for the splash screen 
files.

Can I apply the patch? Please CC me as I am not subscribed to this list.

Regards,
Ravi

Index: src/splashscreen.cpp
===================================================================
RCS file: /home/kde/kdevelop/src/splashscreen.cpp,v
retrieving revision 1.11
diff -u -p -w -r1.11 splashscreen.cpp
--- src/splashscreen.cpp        23 May 2003 02:14:09 -0000      1.11
+++ src/splashscreen.cpp        13 Sep 2003 17:10:21 -0000
@@ -19,8 +19,8 @@
 SplashScreen::SplashScreen()
   : QObject()
 {
-  m_splash = new QWidget( TopLevel::getInstance()->main(),
-    "splash", WStyle_NoBorder | WStyle_Customize | WStyle_Tool );
+  m_splash = new QWidget( 0,
+    "splash", WStyle_NoBorder | WStyle_Customize | WStyle_Tool | 
WStyle_StaysOnTop );

   QVBoxLayout *vbox = new QVBoxLayout(m_splash);

Comment 3 Lubos Lunak 2003-09-16 11:10:30 UTC

Please simply use Q/KSplashScreen.

Comment 4 Roger Larsson 2003-09-24 01:20:18 UTC

OK, splash is now placed in center of screen. 
resolution: REMIND    for the use of Q/KSplashScreen

Comment 5 Amilcar do Carmo Lucas 2004-01-21 15:10:06 UTC

Reopen, because the REMIND state "hides" the bug

Comment 6 Jens Dagerbo 2004-01-21 15:17:57 UTC

Now that we're targetting Qt-3.2/KDE-3.2 we should really switch to QSplashScreen, like Lubos suggests in comment #3. We'll need an updated graphic though.

Comment 7 Roger Larsson 2004-04-26 23:58:32 UTC

Graphic looking greenish now - bug fixed?

Comment 8 Jens Dagerbo 2004-06-17 00:36:04 UTC

KDevelop HEAD now uses QSplashScreen (KSplashScreen is presently too buggy)

So.. I guess FIXED. :)