Merge lp:~seb128/dialer-app/insert-char-cursor into lp:dialer-app

[Sebastien Bacher]

Insert '+' where the cursor is located

Insert '+' where the cursor is located

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:409
http://jenkins.qa.ubuntu.com/job/dialer-app-ci/575/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dialer-app-vivid-i386-ci/88
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/3032
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/2577
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/3030
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/3030/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/20870

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/dialer-app-ci/575/rebuild

review: Approve (continuous-integration)

[Michael Zanetti (mzanetti)]

Tested it. Works fine now. Thanks. I'll leave top-approving to someone deeper involved in the project.

review: Approve

[Gustavo Pichorim Boiko (boiko)]

Looks good!

review: Approve

=== modified file 'src/qml/DialerPage/DialerPage.qml'
--- src/qml/DialerPage/DialerPage.qml	2015-04-08 22:41:26 +0000
+++ src/qml/DialerPage/DialerPage.qml	2015-06-01 13:48:25 +0000
@@ -376,8 +376,8 @@
                     mainView.callVoicemail()
                 } else if (keycode == Qt.Key_0) {
                     // replace 0 by +
-                    dialNumber = dialNumber.substring(0, dialNumber.length - 1)
-                    dialNumber += "+"
+                    input.remove(input.cursorPosition - 1, input.cursorPosition)
+                    input.insert(input.cursorPosition, "+")
                 }
             }
         }