Re: [PATCH manager] fix #6211: ui: vm: options: add tooltip to protection option

["Elias Huhsovitz" <e.huhsovitz@proxmox.com>]

I tried moving the icon to the `boxLabel` as suggested, but in my opinion, it looks a bit off.

Instead of adding a tooltip to clarify the ambiguous "Protection" label, I propose an alternative design decision.
We could rename the option entirely to "Disable VM and Disk removal".
This would make the option self-explanatory and allow us to omit the tooltip altogether.

Let me know your thoughts on this approach. 

On Wed Jul 15, 2026 at 2:17 PM CEST, Dominik Csapak wrote:
> would speak anything against putting the icon into the 'boxLabel'
> instead of to the fieldlabel?
>
> that would put it right next to the actual checkbox and not
> to the relatively generic 'enabled' label
> i think our users should now what 'enabled' means, but
> what we want to convey is the meaning of the actual option
>
> On 7/13/26 1:47 PM, Elias Huhsovitz wrote:
>> The "protection" options was ambiguous.
>> Add tooltip to briefly describe the option.
>> 
>> Signed-off-by: Elias Huhsovitz <e.huhsovitz@proxmox.com>
>> ---
>>   www/manager6/qemu/Options.js | 4 ++++
>>   1 file changed, 4 insertions(+)
>> 
>> diff --git a/www/manager6/qemu/Options.js b/www/manager6/qemu/Options.js
>> index 8a4721a0..9ade5ccb 100644
>> --- a/www/manager6/qemu/Options.js
>> +++ b/www/manager6/qemu/Options.js
>> @@ -326,6 +326,10 @@ Ext.define('PVE.qemu.Options', {
>>                                 defaultValue: 0,
>>                                 deleteDefaultValue: true,
>>                                 fieldLabel: gettext('Enabled'),
>> +                              afterLabelTextTpl: ' <i class="fa fa-question-circle" style="color: #888; cursor: help; margin-left: 4px;"></i>',
>> +                              autoEl: {
>> +                                  'data-qtip': gettext('Prevents the VM and its disks from being removed')
>> +                              }
>>                             },
>>                         }
>>                       : undefined,