Merge lp:~mikemc/ubuntuone-credentials/add-input-hints into lp:ubuntuone-credentials

Proposed by Mike McCracken
Status: Merged
Approved by: dobey
Approved revision: 67
Merged at revision: 67
Proposed branch: lp:~mikemc/ubuntuone-credentials/add-input-hints
Merge into: lp:ubuntuone-credentials
Diff against target: 43 lines (+4/-0)
3 files modified
online-accounts-provider/LoginForm.qml (+1/-0)
online-accounts-provider/NewAccount.qml (+1/-0)
online-accounts-provider/RegisterForm.qml (+2/-0)
To merge this branch: bzr merge lp:~mikemc/ubuntuone-credentials/add-input-hints
Reviewer Review Type Date Requested Status
dobey (community) Approve
PS Jenkins bot continuous-integration Approve
Roberto Alsina (community) Approve
Review via email: mp+189364@code.launchpad.net

Commit message

- Use email-specific keyboard layout and avoid storing passwords in predictive input dictionary (LP: #1235282)

Description of the change

- Use email-specific keyboard layout and avoid storing passwords in predictive input dictionary (LP: #1235282)

Requires testing on device.

bzr branch lp:~mikemc/ubuntuone-credentials/add-input-hints
cd add-input-hints
mkdir build
cd build
cmake -DLIB_SUFFIX=/arm-linux-gnueabihf/ -DCMAKE_INSTALL_PREFIX=/usr/ ..
make
sudo make install

Then run system-settings, click "accounts", add a new U1 account and note the keyboard layout used for the email field. It should have an '@' sign next to the space bar.

I'm not aware of a way to test the sensitiveData flag.

To post a comment you must log in.
Revision history for this message
Roberto Alsina (ralsina) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
dobey (dobey) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'online-accounts-provider/LoginForm.qml'
2--- online-accounts-provider/LoginForm.qml 2013-09-24 20:26:26 +0000
3+++ online-accounts-provider/LoginForm.qml 2013-10-04 16:24:11 +0000
4@@ -27,6 +27,7 @@
5 Keys.onReturnPressed: main.processForm();
6 KeyNavigation.tab: twoFactorUI.visible ? twoFactorTextField : emailTextField /* from main */
7 KeyNavigation.backtab: emailTextField /* from main */
8+ inputMethodHints: Qt.ImhSensitiveData
9 }
10
11 Label {
12
13=== modified file 'online-accounts-provider/NewAccount.qml'
14--- online-accounts-provider/NewAccount.qml 2013-09-24 20:26:26 +0000
15+++ online-accounts-provider/NewAccount.qml 2013-10-04 16:24:11 +0000
16@@ -85,6 +85,7 @@
17 focus: true
18 KeyNavigation.tab: loginForm.visible ? loginForm.passwordTextField : registerForm.nameTextField
19 KeyNavigation.backtab: loginForm.visible ? ( loginForm.twoFactorVisible ? loginForm.twoFactorTextField : loginForm.passwordTextField) : registerForm.confirmPasswordTextField
20+ inputMethodHints: Qt.ImhEmailCharactersOnly
21 }
22
23 Row {
24
25=== modified file 'online-accounts-provider/RegisterForm.qml'
26--- online-accounts-provider/RegisterForm.qml 2013-09-24 20:26:26 +0000
27+++ online-accounts-provider/RegisterForm.qml 2013-10-04 16:24:11 +0000
28@@ -34,6 +34,7 @@
29 width: main.width - (2 * main.anchors.margins)
30 KeyNavigation.tab: confirmPasswordTextField
31 KeyNavigation.backtab: nameTextField
32+ inputMethodHints: Qt.ImhSensitiveData
33 }
34
35 TextField {
36@@ -44,6 +45,7 @@
37 width: main.width - (2 * main.anchors.margins)
38 KeyNavigation.tab: emailTextField /* from main */
39 KeyNavigation.backtab: passwordTextField
40+ inputMethodHints: Qt.ImhSensitiveData
41 }
42
43 Row {

Subscribers

People subscribed via source and target branches

to all changes: