Bug 242238

Summary:	Conflicts accelerator keys for "Constants Buttons" and "Configure KCalc"
Product:	[Applications] kcalc	Reporter:	Funda Wang <fundawang>
Component:	general	Assignee:	Evan Teran <evan.teran>
Status:	RESOLVED FIXED
Severity:	normal	CC:	cfeck, evan.teran, lueck
Priority:	NOR
Version:	2.0
Target Milestone:	---
Platform:	Mandriva RPMs
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Funda Wang 2010-06-20 11:05:32 UTC

Version:           2.0 (using Devel) 
OS:                Linux

Both "Constants Buttons" and "Configure KCalc" in Settings menu are using C as their accelerator keys. Considering "Configure KCalc" Comes from kdelibs, please change "Constants Buttons"'s accelerator key to another one.

Reproducible: Always

Comment 1 Christoph Feck 2010-10-26 03:48:18 UTC

I don't see both using C; the "Configure KCalc" gets assigned a K.

Comment 2 Funda Wang 2010-10-26 05:21:16 UTC

$ cat kdeutils/kcalc/kcalc.cpp |grep Buttons
action_constants_show_->setText(i18n("&Constants Buttons"));

$ cat kdelibs/kdeui/actions/kstandardaction_p.h|grep Configure
{ Preferences,   KStandardShortcut::Preferences, "options_configure", I18N_NOOP("&Configure %1..."), 0, "configure" },

The reason you are seeing 'Configure &KCalc' is that kdelibs will try to solve conflict accelerator keys. But for translators, especially non-latin languages like Chinese or Japanese, we cannot tell which key will be used at that time.

Comment 3 Burkhard Lück 2011-06-25 08:36:23 UTC

Would changing 
action_constants_show_->setText(i18n("&Constants Buttons"));
to
action_constants_show_->setText(i18n("Constants &Buttons"));

solve this issue?

Comment 4 Evan Teran 2012-04-10 20:27:31 UTC

I've applied a patch which does exactly what is proposed in comment 3 to the 4.8 branch.