<table><tr><td style="">rthomas created this revision.<br />rthomas added a reviewer: jgrulich.<br />rthomas added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />Herald added 1 blocking reviewer(s): jgrulich.<br />rthomas requested review of this revision.
</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/D18689">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This bug has already been reported in bug 397352</p>

<p>Whenever I'm adding a bridge interface to my system, plasma-nm changes the connection icon to a disconnected state even though an active wired or wireless connection is present. In my case, even though I have an active WiFi connection running in the background, as soon as I add the bridge interface, the connection icon changes from 'network-wireless-100' to 'network-wireless-available'.</p>

<p>This is probably happening because when the bridge becomes active, NetworkManager considers it the 'ActivatingConnection' and we are choosing the icon based on the 'ActivatingConnection' without running any checks on it. If the 'ActivatingConnection' returns an invalid object we move onto the 'PrimaryConnection'. I have made this diff to only consider the ActivatingConnection for the icon if the first Device for that connection is a "Wifi", "Ethernet", "Modem" or "Bluetooth". If it's not then move to the PrimaryConnection.</p>

<p>BUG: 397352</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R116 Plasma Network Management Applet</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18689">https://phabricator.kde.org/D18689</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>libs/declarative/connectionicon.cpp</div></div></div><br /><div><strong>To: </strong>rthomas, jgrulich<br /><strong>Cc: </strong>plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>