broulik created this revision.
broulik added reviewers: Plasma, sitter.
broulik set the repository for this revision to rPLASMAWORKSPACE Plasma 
Workspace.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  When a device notification arrives, such as "you can now safely unmount" or 
"failed to unmount" device notifier would unconditionally open showing an empty 
list.
  Since we now show device notifications only connected to the device they're 
about, it makes no sense to expand it for invisible devices. Applications where 
mounting or unmounting can fail for non-removable devices, like Dolphin, 
already show the error message within the UI anyway, making the popup redundant.
  
  BUG: 368894
  FIXED-IN: 5.8.0

TEST PLAN
  - plugged in USB key, still got popup
  - unmounted USB key from Dolphin, devicenotifier opened, telling me I could 
safely remove it now
  - tried to unmount / from Dolphin, got a "not authorized" error in Dolphin 
and no longer got an empty device notifier popup
  
  Didn't actually test the snappy usecase described in the bug report.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

AFFECTED FILES
  applets/devicenotifier/package/contents/ui/devicenotifier.qml

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

To: broulik, #plasma, sitter
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas

Reply via email to