Bug 242238 - Conflicts accelerator keys for "Constants Buttons" and "Configure KCalc"
Summary: Conflicts accelerator keys for "Constants Buttons" and "Configure KCalc"
Status: RESOLVED FIXED
Alias: None
Product: kcalc
Classification: Applications
Component: general (show other bugs)
Version: 2.0
Platform: Mandriva RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Evan Teran
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-20 11:05 UTC by Funda Wang
Modified: 2013-02-08 15:18 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

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