firecat53 / networkmanager-dmenu

Control NetworkManager via dmenu
MIT License
783 stars 74 forks source link

add section for VLAN connections #128

Closed ndwarshuis closed 7 months ago

ndwarshuis commented 9 months ago

seems like an obvious addition

vlan entries will appear like "ID:VLAN" between access point entries and vpn entries

firecat53 commented 7 months ago

I appreciate the contribution, however, I need you to remove all of the formatting changes so I (and everyone else) can actually see the "meat" of the PR. My formatting style doesn't necessarily match what you changed :smile:

Let me know if it's easier to resubmit as a new PR or change the existing PR.

Thanks!

ndwarshuis commented 7 months ago

@firecat53 sorry, I had black turned on and it autoformatted. much smaller now

firecat53 commented 7 months ago

Thanks!