← Back to team overview

ubuntu-touch-coreapps-reviewers team mailing list archive

[Merge] lp:~gang65/ubuntu-calculator-app/ubuntu-calculator-app-display-cursor into lp:ubuntu-calculator-app

 

Bartosz Kosiorek has proposed merging lp:~gang65/ubuntu-calculator-app/ubuntu-calculator-app-display-cursor into lp:ubuntu-calculator-app.

Commit message:
Show cursor after calculator run and after keyboard swiping.



Requested reviews:
  Jenkins Bot (ubuntu-core-apps-jenkins-bot): continuous-integration
  Ubuntu Calculator Developers (ubuntu-calculator-dev)

For more details, see:
https://code.launchpad.net/~gang65/ubuntu-calculator-app/ubuntu-calculator-app-display-cursor/+merge/277359

Show cursor after calculator run and after keyboard swiping.

-- 
Your team Ubuntu Calculator Developers is requested to review the proposed merge of lp:~gang65/ubuntu-calculator-app/ubuntu-calculator-app-display-cursor into lp:ubuntu-calculator-app.
=== modified file 'app/ubuntu-calculator-app.qml'
--- app/ubuntu-calculator-app.qml	2015-10-28 20:56:47 +0000
+++ app/ubuntu-calculator-app.qml	2015-11-12 15:07:41 +0000
@@ -38,7 +38,6 @@
 
     width: units.gu(40);
     height: units.gu(70);
-    focus: true
 
     // This is our engine
     property var mathJs: MathJs.mathJs;

=== modified file 'app/ui/CalcKeyboard.qml'
--- app/ui/CalcKeyboard.qml	2015-06-21 22:59:38 +0000
+++ app/ui/CalcKeyboard.qml	2015-11-12 15:07:41 +0000
@@ -54,6 +54,7 @@
             currentIndex = index;
             snapAnimation.to = index * (flickableKeyboard.width)
             snapAnimation.start()
+            textInputField.focus = true
         }
 
         UbuntuNumberAnimation {