<table><tr><td style="">cfeck added a reviewer: teran.<br />cfeck added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D12099">View Revision</a></tr></table><br /><div><div><p>I like the idea. I remember calculators that enabled "repeated operations" automatically when the operation key was pressed twice (and cleared the mode when pressed again). A 'K' was displayed (whatever that meant) next to the memory indicator, whenever the repeated mode was activated.</p>

<p>What I am unsure about is the label of the checkbox. It could mention "Operation" instead of "Action", and the wording "Equal Press" sounds confusing due to word classes. My suggestion for the label would be something like "Repeat Operation for each Result". Maybe :)</p>

<p>The setting is not saved, is it? I guess users would expect any setting from the settings dialog to not be temporary, unlike maybe options that are toggled via the menus.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D12099#inline-60945">View Inline</a><span style="color: #4b4d51; font-weight: bold;">kcalc_core.cpp:846</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #aa4000">if</span> <span class="p">(</span><span class="n">KCalcSettings</span><span style="color: #aa2211">::</span><span class="n">repeatLastOperation</span><span class="p">()){</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">      <span style="color: #aa4000">if</span> <span class="p">(</span><span class="n">func</span> <span style="color: #aa2211">!=</span> <span class="n">FUNC_EQUAL</span> <span style="color: #aa2211">&&</span> <span class="n">func</span> <span style="color: #aa2211">!=</span> <span class="n">FUNC_PERCENT</span><span class="p">)</span> <span class="p">{</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Space before '{'</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R353 KCalc</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12099">https://phabricator.kde.org/D12099</a></div></div><br /><div><strong>To: </strong>mschiller, KDE Applications, teran<br /><strong>Cc: </strong>cfeck, kde-utils-devel<br /></div>