<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, sitter.<br />broulik set the repository for this revision to rPLASMAWORKSPACE Plasma Workspace.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D2823" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>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.<br />
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.</p>

<p>BUG: 368894<br />
FIXED-IN: 5.8.0</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">plugged in USB key, still got popup</li>
<li class="remarkup-list-item">unmounted USB key from Dolphin, devicenotifier opened, telling me I could safely remove it now</li>
<li class="remarkup-list-item">tried to unmount / from Dolphin, got a "not authorized" error in Dolphin and no longer got an empty device notifier popup</li>
</ul>

<p>Didn't actually test the snappy usecase described in the bug report.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMAWORKSPACE Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2823" rel="noreferrer">https://phabricator.kde.org/D2823</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/devicenotifier/package/contents/ui/devicenotifier.qml</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, sitter<br /><strong>Cc: </strong>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>