Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hey, I made some modifications to the script to fix some "bugs" and modify some behaviour.
Remove
More Options
This way all the subcommands of more options are directly searchable and one does not need to go through an extra menu to, for example, activate a VPN connection.
Update configuration place & order
Updated the configuration parsing order to check for configuration in
In the stated order. The reason is that user configuration should take precedence over the default configuration
and moving the configuration to a specific directory is to avoid any potential conflict with rofi configurations.
Clicking on the column header
SSID SECURITY
does not close the programrofi-network-manager restarts when the column header is clicked instead of closing.
Format & spread out code
I found that some lines where very long and/or hard to read, so I split them and applied
shfmt
to the entire file.