Bug 54497 - kooka configure toolbar weirdness (see comment #2)
Summary: kooka configure toolbar weirdness (see comment #2)
Status: RESOLVED FIXED
Alias: None
Product: kooka
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: ---
Assignee: Klaas Freitag
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-02-11 21:17 UTC by Michael R Tharp
Modified: 2004-05-14 16:20 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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