Merge lp:~rpadovani/ubuntu-calculator-app/tempResultOnlyInParenthesis into lp:ubuntu-calculator-app

[Riccardo Padovani]

Remove * from operators with priority

We try to help users calculating temporary results. Unfortunately at the moment the feature is confusing, 'cause we calculate the temporary result just after a * operator.

So if the users write
3*3^3
we print
9^3

Then user presses = and we print 27 (that is right) but is confusing because 9^3 = 81.

So we can calculate temporary result only in the parenthesis, cause nothing has priorty > than parenthesis

[Jenkins Bot (ubuntu-core-apps-jenkins-bot)]

PASSED: Continuous integration, rev:249
https://core-apps-jenkins.ubuntu.com/job/calculator-app-ci/37/
Executed test runs:
    None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/218/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/calculator-app-ci/37/rebuild

review: Approve (continuous-integration)

[Bartosz Kosiorek (gang65)]

Work ok for me.

review: Approve

=== modified file 'app/ubuntu-calculator-app.qml'
--- app/ubuntu-calculator-app.qml	2015-11-28 20:44:02 +0000
+++ app/ubuntu-calculator-app.qml	2015-12-03 16:45:49 +0000
@@ -182,7 +182,7 @@
         }
 
         // Add here operators that have always priority
-        if ((visual.toString() === "*") || (visual.toString() === ")")) {
+        if (visual.toString() === ")") {
             isFormulaIsValidToCalculate = true;
         }
     }