Bug 54497

Summary:	kooka configure toolbar weirdness (see comment #2)
Product:	[Applications] kooka	Reporter:	Michael R Tharp <chexmeex>
Component:	general	Assignee:	Klaas Freitag <freitag>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Unlisted Binaries
OS:	Linux
Latest Commit:		Version Fixed In:

Description Michael R Tharp 2003-02-11 21:17:59 UTC

Version:            (using KDE KDE 3.1)
Installed from:    Unspecified Linux
Compiler:          gcc (GCC) 3.2 20020903 (Red Hat Linux 8.0 3.2-7) 
OS:          Linux

KDE release 3.1.0, kooka 0.39 (included with release)

Settings --> Configure Toolbars...

It shows both "Show Scan Parameter" and "Show Scan Parameters" in the available actions.

If you choose "Show Scan Parameter" (the one without the 's' on the end), and click it on the toolbar, it immediately crashes signal 11.  The "Show Scan Parameters" option works correctly as far as I can tell.

Comment 1 George Staikos 2003-02-13 06:40:02 UTC

Subject: KDE_3_1_BRANCH: kdegraphics/kooka

CVS commit by staikos: 

GUI: Fix #54497

CCMAIL: 54497-done@bugs.kde.org, freitag@suse.de


  M +5 -3      kooka.cpp   1.35.2.1


--- kdegraphics/kooka/kooka.cpp  #1.35:1.35.2.1
@@ -119,7 +119,9 @@ void Kooka::setupActions()
     m_statusbarAction = KStdAction::showStatusbar(this, SLOT(optionsShowStatusbar()),
                                                   actionCollection());
-    m_scanParamsAction = new KToggleAction(i18n("Show Scan &Parameter"), 0, this,
-                                           SLOT(optionsShowScanParams()), actionCollection(),
-                                           "show_scanparams" );
+    m_scanParamsAction = new KToggleAction(i18n("Show Scan &Parameters"),
+                                           KShortcut(), this,
+                                           SLOT(optionsShowScanParams()),
+                                           actionCollection(),
+                                           "dock_scanparams" );
     m_scanParamsAction->setChecked( true );

Comment 2 Michael R Tharp 2003-02-13 19:46:19 UTC

This fixes the crash, but leaves two "Show Scan Parameters" Available actions in
the listbox.  When you click the "Show Sca..." option and click the right arrow
to put it on the toolbar, only one is displayed.  The duplicate option
disappears from the listbox.  When you remove it from the toolbar, there are two
again.

Comment 3 Michael R Tharp 2003-02-13 20:21:28 UTC

see comment #2

Comment 4 Klaas Freitag 2004-05-14 16:20:05 UTC

Removed duplicate creation of the action, that fixes the issue described in comment 2