Re: [PATCH datacenter-manager] ui: configuration: slightly clean up tool tips for subscrion registry

["Lukas Wagner" <l.wagner@proxmox.com>]

On Tue May 26, 2026 at 4:30 PM CEST, Shannon Sterz wrote:
> * properly quote actions
> * rephrase some tips slightly for clarity.
>
> Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
> ---
>  ui/src/configuration/subscription_keys.rs     | 6 +++---
>  ui/src/configuration/subscription_registry.rs | 8 ++++----
>  2 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/ui/src/configuration/subscription_keys.rs b/ui/src/configuration/subscription_keys.rs
> index 5f3e9aa..c35a546 100644
> --- a/ui/src/configuration/subscription_keys.rs
> +++ b/ui/src/configuration/subscription_keys.rs
> @@ -286,7 +286,7 @@ impl LoadableComponent for SubscriptionKeyGridComp {
>                          .on_activate(link.change_view_callback(|_| Some(ViewState::Add))),
>                  )
>                  .tip(tr!(
> -                    "Add one or more subscription keys to the pool; the Assign step \
> +                    "Add one or more subscription keys to the pool; the 'Assign' step \
>                       happens later."
>                  )),
>              )
> @@ -299,7 +299,7 @@ impl LoadableComponent for SubscriptionKeyGridComp {
>                          .on_activate(link.change_view_callback(|_| Some(ViewState::Assign))),
>                  )
>                  .tip(tr!(
> -                    "Pin the selected key to a remote node; Apply Pending pushes the \
> +                    "Pin the selected key to a remote node; 'Apply Pending' pushes the \
>                       assignment to the remote."
>                  )),
>              )
> @@ -325,7 +325,7 @@ impl LoadableComponent for SubscriptionKeyGridComp {
>                  )
>                  .tip(tr!(
>                      "Propose a one-key-per-node assignment for nodes that have no active \
> -                     subscription, then queue it pending Apply."
> +                     subscription, then queue applying it."
>                  )),

Maybe

 "Propose a key assignment for nodes that have no active subscription,
 allowing to queue it for applying the proposal later."

could be a tad more clear? Although it might be a bit long for a
tooltip...

The remaining changes look good to me, I think quoting the 'action'
looks nicer in the tooltip.

Reviewed-by: Lukas Wagner <l.wagner@proxmox.com>