[Differential] [Commented On] D3257: Close the system tray on escape - system tray

[davidedmundson (David Edmundson)]

davidedmundson added inline comments.

INLINE COMMENTS

> mart wrote in PlasmoidPopupsContainer.qml:30
> wouldn't this break keyboard navigation in an item if supported? (like going up and down in the device list in the device notifier)

It shouldn't. See paragraph 4 of description.

However, when testing I can't get keyboard nav of device notifier to work pre-patch, and it continuued not working after this patch.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma
Cc: mart, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161104/ae5c29ad/attachment.html>