ubuntu-touch-coreapps-reviewers team mailing list archive

[Victor Thompson <victor.thompson@xxxxxxxxx>]

Victor Thompson has proposed merging lp:~vthompson/music-app/refactor-force-focus-dialogs into lp:music-app/refactor.

Commit message:
Force active focus to dialogs

Requested reviews:
  Music App Developers (music-app-dev)
Related bugs:
  Bug #1435260 in Ubuntu Music App: "TextFields in most dialogs should be focused"
  https://bugs.launchpad.net/music-app/+bug/1435260

For more details, see:
https://code.launchpad.net/~vthompson/music-app/refactor-force-focus-dialogs/+merge/253912

Force active focus to dialogs
-- 
Your team Music App Developers is requested to review the proposed merge of lp:~vthompson/music-app/refactor-force-focus-dialogs into lp:music-app/refactor.

=== modified file 'app/components/Dialog/EditPlaylistDialog.qml'
--- app/components/Dialog/EditPlaylistDialog.qml	2015-02-04 20:35:53 +0000
+++ app/components/Dialog/EditPlaylistDialog.qml	2015-03-24 03:55:05 +0000
@@ -77,4 +77,6 @@
         color: styleMusic.dialog.cancelButtonColor
         onClicked: PopupUtils.close(dialogEditPlaylist)
     }
+
+    Component.onCompleted: playlistName.forceActiveFocus()
 }

=== modified file 'app/components/Dialog/NewPlaylistDialog.qml'
--- app/components/Dialog/NewPlaylistDialog.qml	2015-02-04 20:35:53 +0000
+++ app/components/Dialog/NewPlaylistDialog.qml	2015-03-24 03:55:05 +0000
@@ -72,4 +72,6 @@
         color: styleMusic.dialog.cancelButtonColor
         onClicked: PopupUtils.close(dialogNewPlaylist)
     }
+
+    Component.onCompleted: playlistName.forceActiveFocus()
 }