Code review comment for lp:~rpadovani/ubuntu-calculator-app/closeBrackets150127

Revision history for this message
Giulio Collura (gcollura) wrote :

I totally support rpadovani's idea. Since this calculator app doesn't want to be a scientific calculator (btw, my scientific calc, the one I was during the exams, auto-closes brackets on short expressions), auto-closing brackets is a must-have feature.
While other kind of errors, like too many operators or too many brackets, must be marked as 'Syntax errors', maybe by highlighting with an animation the current calculation (without popover/popups that may annoy the user).
This is my point of view :)
You have my approval on this branch!

« Back to merge proposal