https://bugs.kde.org/show_bug.cgi?id=57349
Albert Astals Cid <aa...@kde.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|CONFIRMED |RESOLVED Latest Commit| |https://commits.kde.org/kca | |lc/b0ce4ff01b5e1f7f4c2340b9 | |0f326491437c076c --- Comment #23 from Albert Astals Cid <aa...@kde.org> --- Git commit b0ce4ff01b5e1f7f4c2340b90f326491437c076c by Albert Astals Cid, on behalf of Maximilian Schiller. Committed on 05/10/2018 at 21:09. Pushed by aacid into branch 'master'. Corrected behaviour on repeated pressing of operator buttons Summary: Fixes "wrong" behavior to the correct one as described in the bug report. Before the fix pressing "10 * / 5 =" normally would result in 20 (Stack: (10,*), (10,/), (5,=)) but with those changes it results in 2 (Stack: (10, /), (5, =)). Internally this is achieved by "blocking" node adding after a operator (*,-,/,...) has been pressed until a number is entered. If another operator is pressed whilst blocked the top (last) operator is changed instead. Reviewers: cfeck, teran, #kde_applications Reviewed By: cfeck Subscribers: aacid, cfeck, kde-utils-devel Differential Revision: https://phabricator.kde.org/D15463 M +3 -1 kcalc.cpp M +17 -3 kcalc_core.cpp M +3 -0 kcalc_core.h https://commits.kde.org/kcalc/b0ce4ff01b5e1f7f4c2340b90f326491437c076c -- You are receiving this mail because: You are watching all bug changes.