[pve-devel] [PATCH proxmox_login_manager] ui: Changed "sites" to "hosts".

[Dominik Csapak]

On 5/28/25 13:04, Alexander Abraham wrote:
> Changed the "Available Sites" text inside the added PVE
> hosts view to "Available Hosts" because "Sites" is a bit
> odd.
> 

I disagree. While it could happen that users add all hosts to their app,
every entry in the app is basically a connection to a cluster, which can contain
multiple hosts.

I know we don't have a fallback connection mechanism for other hosts yet, but
IMO  'Sites' / 'Available Sites' still fit better as we don't list all
hosts from all clusters here.

(The same holds for the other patch for pve_flutter_frontend)

> Signed-off-by: Alexander Abraham <a.abraham at proxmox.com>
> ---
>   lib/proxmox_login_selector.dart | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/proxmox_login_selector.dart b/lib/proxmox_login_selector.dart
> index fb1218b..7fc0068 100644
> --- a/lib/proxmox_login_selector.dart
> +++ b/lib/proxmox_login_selector.dart
> @@ -127,7 +127,7 @@ class _ProxmoxLoginSelectorState extends State<ProxmoxLoginSelector> {
>                 const Padding(
>                   padding: EdgeInsets.all(12.0),
>                   child: Text(
> -                  'Available Sites',
> +                  'Available Hosts',
>                     style: TextStyle(
>                       fontSize: 18,
>                       fontWeight: FontWeight.bold,