Bug 93535

Summary:	'menu/settings/show statusbar' behaves erroneusly
Product:	[Applications] umbrello	Reporter:	Piotr Michalczyk (PL) <p_michalczyk>
Component:	general	Assignee:	Umbrello Development Group <umbrello-devel>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version First Reported In:	1.3.1
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Piotr Michalczyk (PL) 2004-11-18 20:51:35 UTC

Version:           1.3.1 (using KDE 3.2.3,  (testing/unstable))
Compiler:          gcc version 3.3.3 (Debian 20040422)
OS:                Linux (i686) release 2.6.7

When 'umbrello' is started the 'settings/show statusbar' menu item is unchecked, while actually the statusbar is visible.
Moreover statusbar visibility isn't preserved between app launches: when i hide it, then close and reopen 'umbrello' the situation is as above (item - unchecked, statusbar visible).

Comment 1 Jonathan Riddell 2004-11-19 17:13:16 UTC

Is this even a useful option to have, can we just remove it?

Comment 2 Piotr Michalczyk (PL) 2004-11-21 20:20:21 UTC

For people with small (14") monitors (like me), who work on low screen resolutins (800x600) it is important to use available space efficiently. Few pixels here, few there, hidden statusbar, menubar moved to the left/right (instead of top) and you get quite big area for editing diagram.
That's why I thing it would be better to remove statusbar, than to remove this menu option while leaving statusbar.

Comment 3 Jonathan Riddell 2004-11-22 16:51:05 UTC

CVS commit by jriddell: 

Fix hide/show statusbar
beastie 93535: 'menu/settings/show statusbar' behaves erroneusly 
BUG:93535


  M +5 -5      uml.cpp   1.120


--- kdesdk/umbrello/umbrello/uml.cpp  #1.119:1.120
@@ -84,9 +84,8 @@ UMLApp::UMLApp(QWidget* , const char* na
         // call inits to invoke all other construction parts
         readOptionState();
-        initActions();
         initDocument();
+        initActions(); //now calls initStatusBar() because it is effected by setupGUI();
         initView();
         initClip();
-        initStatusBar();
         readOptions();
         ///////////////////////////////////////////////////////////////////
@@ -269,6 +268,8 @@ void UMLApp::initActions() {
         KStdAction::tipOfDay( this, SLOT( tipOfTheDay() ), actionCollection() );
 
-        // use the absolute path to your umbrelloui.rc file for testing purpose in createGUI();
-        createGUI();
+        initStatusBar(); //call this here because status bar is shown/hidden by setupGUI()
+
+        // use the absolute path to your umbrelloui.rc file for testing purpose in setupGUI();
+        setupGUI();
 
         QPopupMenu* menu = findMenu( menuBar(), QString("settings") );
@@ -358,5 +359,4 @@ void UMLApp::initStatusBar() {
 
         connect(m_doc, SIGNAL( sigWriteToStatusBar(const QString &) ), this, SLOT( slotStatusMsg(const QString &) ));
-        statusBar()->show(); //needs to be forced to show when on first ever startup for some reason
 }
 ////////////////////////////////////////////////////////////////////////////////////////////////////