https://bugs.kde.org/show_bug.cgi?id=516712

--- Comment #11 from Nate Graham <[email protected]> ---
So this is a somewhat sad situation.

Prior to my change in
https://invent.kde.org/plasma/plasma-nm/-/commit/e85e1c0466b30b8d291d7765675044ad3c0fbf92,
the code was ignoring bridges and VLANs and going into setDisconnectedIcon(),
which was already wrong, since there was in fact an active connection.

However inside setDisconnectedIcon(), bridges and VLANs count as wired
networks, so we'd end up in
`setConnectionIcon(QStringLiteral("network-wired-available"));` And that icon
happens to look like a monitor with a wire in the Breeze icon theme.

In e85e1c0466b30b8d291d7765675044ad3c0fbf92 I fixed this to return
`network-wired-disconnected`, because this entire codepath is *supposed to* be
for when there's no connection.

So changing that, which was harmless on its own, revealed that plasma-nm is
ignoring connected bridges and VLANs, and going into setDisconnectedIcon() when
it shouldn't. Previously this was OK-ish because the Breeze icon it gave you
didn't *look* disconnected (even though it was still the semantically wrong
icon to return).

I think the right course of action here is to not consider active bridges and
VLANs to be disconnected in the first place. I'll see if I can fix that.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to