elementary / switchboard-plug-network

Switchboard Network Plug
GNU General Public License v3.0
19 stars 24 forks source link

When ethernet is unplugged don't display information in wired view #149

Closed worldofpeace closed 3 years ago

worldofpeace commented 5 years ago

screenshot from 2019-01-24 11 22 33

Because this is unplugged there's never going to be information to populate this with and the switch isn't going to effect the physical state. This should probably just show some text asking to connect an ethernet cable.

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/69141013-when-ethernet-is-unplugged-don-t-display-information-in-wired-view?utm_campaign=plugin&utm_content=tracker%2F60754170&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F60754170&utm_medium=issues&utm_source=github).
peteruithoven commented 5 years ago

I'd like to disagree. On my laptop, without build in ethernet port), this Wired option only shows up when I plug in a usb to ethernet adapter. Seeing the status of that Wired hardware is interesting, it can tells me for example whether there is something wrong with the cable for example.

I do think that the contents of the Wired page could be improved in this case.

worldofpeace commented 5 years ago

Sorry but I don't know what I mentioned that you've disagreed to.

On my laptop, without build in ethernet port), this Wired option only shows up when I plug in a usb to ethernet adapter. Seeing the status of that Wired hardware is interesting, it can tells me for example whether there is something wrong with the cable for example.

That makes it sound like I've suggested hiding this screenshot from 2019-01-24 19 20 22

And what I'm suggesting is that the view should also just display that hardware status similar to the wifi view screenshot from 2019-01-24 19 17 28 and make the switch insensitive

peteruithoven commented 5 years ago

Apologies, I misunderstood. Looks like we're actually on the same page.

worldofpeace commented 5 years ago

Nice :smile:

pokgak commented 5 years ago

Aside from displaying the hardware status, I think the toggle switch should also be disabled when there is no device to connect to or when there is no ethernet hardware present.

psukys commented 5 years ago

Aside from displaying the hardware status, I think the toggle switch should also be disabled when there is no device to connect to or when there is no ethernet hardware present.

why?

pokgak commented 5 years ago

Forgot to include picture in my last comment. Here it is: image

Connected using wifi and no ethernet cable connected. A bit confusing to me that I still can toggle ethernet on and it shows "unknown" as IP, as if I somehow screwed my configuration.

worldofpeace commented 5 years ago

Yeah this is what I meant by "make the switch insensitive"

NathanBnm commented 4 years ago

I still have this issue. There is an inconsistency between the switchboard plug and the wingpanel indicator.

I mean in the wingpanel indicator the "Wired" option is disabled, and cannot be activated as long as no ethernet cable is connected.

But in the switchboard plug the "Wired" option is always enabled, even if no ethernet cable is connected. Which is useless because it's displaying an "unknown" status.

In addition to that when you disable the "Wired" option in the switchboard plug and close the settings, it re-enables itself. So it can't be disabled.

I think the "Wired" view should adopt the same behavior than the wingpanel indicator mentionned above.

See the screenshots:

In the wingpanel indicator:

image

In the switchboard plug:

image

Dirli commented 4 years ago

I suggested a fix, but it went unnoticed. Who needs to fix this (although it does not affect the functionality), here is the patch.

https://github.com/Dirli/gpde/blob/master/pantheon-base/switchboard-plug-network/files/2.3.0-fix_unplugged_cable.patch