D16106: Focus handling fixes

[David Edmundson]

davidedmundson accepted this revision.
davidedmundson added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> main.qml:425
> +            if (isFolder) {
> +                folderViewLayer.item.forceActiveFocus();
> +            } else {

I would have expected just folderViewLayer.forceActiveFocus() to work?

the FolderViewLayer item should be the only item within the Loader (which is a focus scope) and you added focus: true to FolderViewLayer

REPOSITORY
  R119 Plasma Desktop

BRANCH
  Plasma/5.14

REVISION DETAIL
  https://phabricator.kde.org/D16106

To: hein, broulik, davidedmundson
Cc: davidedmundson, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181010/94f32fba/attachment.html>