D15680: Allow shortcut keys for both decimal separators point and comma

[Christoph Feck]

cfeck requested changes to this revision.
cfeck added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> kcalc.cpp:374
> +	// add shortcut for the other decimal separator (point or comma)
> +	if (QLocale().decimalPoint() == QLatin1String(".")) {
>  		new QShortcut(Qt::Key_Comma, pbPeriod, SLOT(animateClick()));

`QLocale::decimalPoint()` returns a QChar, not a QString.

REPOSITORY
  R353 KCalc

REVISION DETAIL
  https://phabricator.kde.org/D15680

To: apichlkostner, cfeck, teran
Cc: kde-utils-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20180921/3ed113a1/attachment.html>