Development discussion of WireGuard
 help / color / mirror / Atom feed
* Feature request - Number of tunnel entries in root of tray icon context menu
@ 2021-09-27 21:36 wireguard_mailing_list_user
  2021-09-28 23:16 ` Jason A. Donenfeld
       [not found] ` <163287098886.7.7062947333110889290.14414570@simplelogin.co>
  0 siblings, 2 replies; 4+ messages in thread
From: wireguard_mailing_list_user @ 2021-09-27 21:36 UTC (permalink / raw)
  To: wireguard

Hi,

the hardcoded value is 10 (in ui/tray.go)

Wouldn't it be possible to allow a user to set this value instead of 
using a hardcoded one?

E.g. via:
HKEY_CURRENT_USER\SOFTWARE\WireGuard\TunnelsInRootMenu
Default is 10 and if that registry value doesn't exist, 10 is also the 
default

It should accept values from 0 (display all tunnels) to +x (display at 
maximum x tunnels)

Vertical screen space shouldn't be an issue nowadays and imho the value 
of 10 is a bit too limiting.

Thank you!



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Feature request - Number of tunnel entries in root of tray icon context menu
  2021-09-27 21:36 Feature request - Number of tunnel entries in root of tray icon context menu wireguard_mailing_list_user
@ 2021-09-28 23:16 ` Jason A. Donenfeld
       [not found] ` <163287098886.7.7062947333110889290.14414570@simplelogin.co>
  1 sibling, 0 replies; 4+ messages in thread
From: Jason A. Donenfeld @ 2021-09-28 23:16 UTC (permalink / raw)
  To: wireguard_mailing_list_user; +Cc: WireGuard mailing list

Hi,

I don't want to add a knob for this, but I would take a patch that
uses a smarter heuristic than just "10". Feel free to send patches to
the list. The code in question is here:

https://git.zx2c4.com/wireguard-windows/tree/ui/tray.go#n225

For example, if you can calculate the number of entries that can fit
on the vertical height of the screen, maybe take a third? Or two
fifths?

Jason

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Feature request - Number of tunnel entries in root of tray icon context menu
       [not found] ` <163287098886.7.7062947333110889290.14414570@simplelogin.co>
@ 2021-09-30 11:09   ` wireguard_mailing_list_user
  2021-09-30 16:41     ` Jason A. Donenfeld
  0 siblings, 1 reply; 4+ messages in thread
From: wireguard_mailing_list_user @ 2021-09-30 11:09 UTC (permalink / raw)
  To: WireGuard mailing list

Hi,

Apart from the problem that I'm not that much into go(lang) that I could 
provide such a patch myself...

I just don't see the gain that this way would yield because that would 
just replace one knob with another:
A registry value for a fixed amount of maximum entries or a ratio value 
(e.g.: 0.33 or 0.4). The ratio would require a knob as well, otherwise 
the next person comes and asks how to change it because it doesn't fit 
his needs...

Regards,
Ulf



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Feature request - Number of tunnel entries in root of tray icon context menu
  2021-09-30 11:09   ` wireguard_mailing_list_user
@ 2021-09-30 16:41     ` Jason A. Donenfeld
  0 siblings, 0 replies; 4+ messages in thread
From: Jason A. Donenfeld @ 2021-09-30 16:41 UTC (permalink / raw)
  To: wireguard_mailing_list_user; +Cc: WireGuard mailing list

On Thu, Sep 30, 2021 at 5:15 AM <wireguard_mailing_list_user@stdmail.de> wrote:
> The ratio would require a knob as well

Why?

Anyway, we're not adding knobs for any of this, as already stated. If
you've got a better heuristic than "10", then by all means send a
patch and I'd be happy to apply it. If you don't like "10" and you
don't like percentage of the vertical resolution, then come up with a
different heuristic and make a strong argument for it.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-09-30 16:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-27 21:36 Feature request - Number of tunnel entries in root of tray icon context menu wireguard_mailing_list_user
2021-09-28 23:16 ` Jason A. Donenfeld
     [not found] ` <163287098886.7.7062947333110889290.14414570@simplelogin.co>
2021-09-30 11:09   ` wireguard_mailing_list_user
2021-09-30 16:41     ` Jason A. Donenfeld

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).