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

["Shannon Sterz" <s.sterz@proxmox.com>]

On Tue May 26, 2026 at 4:39 PM CEST, Lukas Wagner wrote:
> 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>

Maybe:

    "Propose a key assignment for nodes that have no active subscription and queue its application."

?