rthomas updated this revision to Diff 50771. rthomas added a reviewer: Plasma. rthomas added a comment.
Should have used '&&' in the if statement instead of '||'. We want to shift to the PrimaryConnection if the Device is not among Wifi, Ethernet, Modem and Bluetooth. Reason for choosing those 4 devices is those are the only devices we set icons for (according to my understanding). REPOSITORY R116 Plasma Network Management Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18689?vs=50752&id=50771 REVISION DETAIL https://phabricator.kde.org/D18689 AFFECTED FILES libs/declarative/connectionicon.cpp To: rthomas, jgrulich, #plasma Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart