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
=== modified file 'plugins/wifi/OtherNetwork.qml'
--- plugins/wifi/OtherNetwork.qml 2014-09-18 16:48:38 +0000
+++ plugins/wifi/OtherNetwork.qml 2014-11-18 19:42:19 +0000
@@ -196,6 +196,7 @@
196 id : networkname196 id : networkname
197 objectName: "networkname"197 objectName: "networkname"
198 inputMethodHints: Qt.ImhNoPredictiveText198 inputMethodHints: Qt.ImhNoPredictiveText
199 Component.onCompleted: forceActiveFocus()
199 }200 }
200201
201 Label {202 Label {
@@ -234,8 +235,7 @@
234 visible: securityList.selectedIndex !== 0235 visible: securityList.selectedIndex !== 0
235 echoMode: passwordVisibleSwitch.checked ?236 echoMode: passwordVisibleSwitch.checked ?
236 TextInput.Normal : TextInput.Password237 TextInput.Normal : TextInput.Password
237 inputMethodHints: passwordVisibleSwitch.checked ?238 inputMethodHints: Qt.ImhNoPredictiveText
238 Qt.ImhHiddenText : Qt.ImhNoPredictiveText;
239 onAccepted: {239 onAccepted: {
240 connectAction.trigger();240 connectAction.trigger();
241 }241 }
@@ -249,6 +249,7 @@
249249
250 CheckBox {250 CheckBox {
251 id: passwordVisibleSwitch251 id: passwordVisibleSwitch
252 activeFocusOnPress: false
252 }253 }
253254
254 Label {255 Label {
255256
=== modified file 'plugins/wifi/PageComponent.qml'
--- plugins/wifi/PageComponent.qml 2014-09-26 12:52:31 +0000
+++ plugins/wifi/PageComponent.qml 2014-11-18 19:42:19 +0000
@@ -137,7 +137,8 @@
137 ListItem.SingleValue {137 ListItem.SingleValue {
138 objectName: "connectToHiddenNetwork"138 objectName: "connectToHiddenNetwork"
139 text: i18n.tr("Connect to hidden network…")139 text: i18n.tr("Connect to hidden network…")
140 progression: true140 visible : (actionGroup.actionObject.valid ?
141 actionGroup.actionObject.state : false)
141 onClicked: {142 onClicked: {
142 otherNetworLoader.source = "OtherNetwork.qml";143 otherNetworLoader.source = "OtherNetwork.qml";
143 PopupUtils.open(otherNetworLoader.item);144 PopupUtils.open(otherNetworLoader.item);

Subscribers

People subscribed via source and target branches