Merge lp:~ken-vandine/ubuntu-system-settings/rtm-1387656 into lp:ubuntu-system-settings/rtm-14.09

Proposed by Ken VanDine
Status: Rejected
Rejected by: Sebastien Bacher
Proposed branch: lp:~ken-vandine/ubuntu-system-settings/rtm-1387656
Merge into: lp:ubuntu-system-settings/rtm-14.09
Diff against target: 43 lines (+5/-3)
2 files modified
plugins/wifi/OtherNetwork.qml (+3/-2)
plugins/wifi/PageComponent.qml (+2/-1)
To merge this branch: bzr merge lp:~ken-vandine/ubuntu-system-settings/rtm-1387656
Reviewer Review Type Date Requested Status
Sebastien Bacher (community) Approve
Review via email: mp+242117@code.launchpad.net

Commit message

[wifi] fix multiple issues in the "Connect to other network" dialog

Description of the change

Fixes the following bugs in the "Connect to other network" dialog:
 * Removes the progression on the entry to the dialog
 * Fixes bug where predictive text would remove the password
 * Stops the "Show password" checkbox from stealing focus from the password textfield

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

thanks

review: Approve
Revision history for this message
Sebastien Bacher (seb128) wrote :

seems like we are not doing more rtm landing, so cleaning the list a bit, that one is rather going to be including with the vivid rebase

Unmerged revisions

940. By Ken VanDine

[wifi] fix multiple issues in the "Connect to other network" dialog

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/wifi/OtherNetwork.qml'
2--- plugins/wifi/OtherNetwork.qml 2014-09-18 16:48:38 +0000
3+++ plugins/wifi/OtherNetwork.qml 2014-11-18 19:42:19 +0000
4@@ -196,6 +196,7 @@
5 id : networkname
6 objectName: "networkname"
7 inputMethodHints: Qt.ImhNoPredictiveText
8+ Component.onCompleted: forceActiveFocus()
9 }
10
11 Label {
12@@ -234,8 +235,7 @@
13 visible: securityList.selectedIndex !== 0
14 echoMode: passwordVisibleSwitch.checked ?
15 TextInput.Normal : TextInput.Password
16- inputMethodHints: passwordVisibleSwitch.checked ?
17- Qt.ImhHiddenText : Qt.ImhNoPredictiveText;
18+ inputMethodHints: Qt.ImhNoPredictiveText
19 onAccepted: {
20 connectAction.trigger();
21 }
22@@ -249,6 +249,7 @@
23
24 CheckBox {
25 id: passwordVisibleSwitch
26+ activeFocusOnPress: false
27 }
28
29 Label {
30
31=== modified file 'plugins/wifi/PageComponent.qml'
32--- plugins/wifi/PageComponent.qml 2014-09-26 12:52:31 +0000
33+++ plugins/wifi/PageComponent.qml 2014-11-18 19:42:19 +0000
34@@ -137,7 +137,8 @@
35 ListItem.SingleValue {
36 objectName: "connectToHiddenNetwork"
37 text: i18n.tr("Connect to hidden network…")
38- progression: true
39+ visible : (actionGroup.actionObject.valid ?
40+ actionGroup.actionObject.state : false)
41 onClicked: {
42 otherNetworLoader.source = "OtherNetwork.qml";
43 PopupUtils.open(otherNetworLoader.item);

Subscribers

People subscribed via source and target branches