D24110: Enable removing multiple devices at once
Daniels Umanovskis
noreply at phabricator.kde.org
Tue Sep 24 11:20:42 BST 2019
umanovskis marked an inline comment as done.
umanovskis added inline comments.
INLINE COMMENTS
> ngraham wrote in devices.cpp:209
> You could rename this function to `removeSelectedDevices()` and handle both single and multi-selection behaviors in it.
Done, looks cleaner now - there's still an if-statement for different dialog messages but less duplication overall.
REPOSITORY
R97 Bluedevil
BRANCH
dev-multiselect
REVISION DETAIL
https://phabricator.kde.org/D24110
To: umanovskis, #vdg, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190924/db53158d/attachment-0001.html>
More information about the Plasma-devel
mailing list