Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Auto select active network #407

Merged
merged 2 commits into from
Oct 8, 2024
Merged

Auto select active network #407

merged 2 commits into from
Oct 8, 2024

Conversation

vjr
Copy link
Member

@vjr vjr commented Sep 29, 2024

Automatically select the first device which is in the activated state.

I typically use the wifi connection on my laptop which also has ethernet, as would most folks.

Whenever I open the networking plug, the first device (wired) is pre-selected and I have to do an extra click to select the wifi device to view or make changes.

Now, the wifi gets pre-selected instead, when opening the plug:

network

If both (or neither) wired and (nor) wireless are activated, then the first (wired) is pre-selected, preserving the current (old) behaviour.

Also, while the plug is open, toggling devices (wired, wireless) updates the selected device.
edit: No longer auto switch while plug is already open.

@vjr vjr self-assigned this Sep 29, 2024
@vjr vjr requested a review from a team September 29, 2024 17:28
Copy link
Contributor

@zeebok zeebok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is nice when you open the plug. I am a little hesitant about it switching while it is open, feels almost unintentional. Additionally, in the scenario I am in the menu want to restart one of the items (VPN, eth, wifi, etc) turning it off moves it away and then I need to move back to turn it on. But that is just a single personal opinion.

@vjr
Copy link
Member Author

vjr commented Oct 8, 2024

I actually agree with you @zeebok so I will remove that connect_after () line so it does not update when the plug is open.

@vjr vjr merged commit b41c3fc into main Oct 8, 2024
4 checks passed
@vjr vjr deleted the auto-select-active-network branch October 8, 2024 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants