ubuntu-touch-coreapps-reviewers team mailing list archive

[Victor Thompson <victor.thompson@xxxxxxxxx>]

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

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/force-focus-dialogs/+merge/253909

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

=== modified file 'common/SongsPage.qml'
--- common/SongsPage.qml	2015-03-19 19:34:36 +0000
+++ common/SongsPage.qml	2015-03-24 02:19:42 +0000
@@ -585,6 +585,8 @@
                 color: styleMusic.dialog.cancelButtonColor
                 onClicked: PopupUtils.close(dialogEditPlaylist)
             }
+
+            Component.onCompleted: playlistName.forceActiveFocus()
         }
     }
 

=== modified file 'music-app.qml'
--- music-app.qml	2015-02-05 22:40:14 +0000
+++ music-app.qml	2015-03-24 02:19:42 +0000
@@ -1079,6 +1079,8 @@
                 color: styleMusic.dialog.cancelButtonColor
                 onClicked: PopupUtils.close(dialogNewPlaylist)
             }
+
+            Component.onCompleted: playlistName.forceActiveFocus()
         }
     }