Jelle van der Waa

Just had this with kpatches, it is shown at the top but it's missing some spacing in my opinion.  Does this PR handle that too?

> @jelly : Thanks for spotting -- I suppose that's another case of missing a `` wrapper. As this is an utterly complex endeavour, I'll split up the cockpit-components-dialog PR...

Found out that cockpit-machines needs fixing: 

The tricky part here is that removing the prefix and making it absolute works for the login page, but not when you for example browser to https://cockpit.lan/system/services/notfound as that would...

In theory we could make the fail.html respect `UrlRoot` by getting the value in cockpit-ws and setting it as a template string in `cockpit_web_response_error` but this is a bit tricky...

> I'm not a huge fan of this proposal. > > Why not just use `git blame -w`? (and `-C -M` for that matter)? True that works for whitespace changes,...

Does this mean VPN's created by networkmanager? Interestingly networkmanager shows my wireguard vpn under "Wireguard" and openvpn under "VPN"

> I'm trying to integrate Wireguard and OpenVPN > > I will start with Wireguard because it's more easy. OpenVPN have a lot of options to deal. Cool! Thanks for...

> Thanks @jelly for your time! > > As do you know cockpit in deep, in OpenVPN scenario We need files with keys. Is there any solution better than specify...

Looks good, I think it makes sense to to first make a PR for just the wireguard functionality and then in future PR's add other VPN solutions. I do wonder...