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