Re: [PATCH proxmox-backup v5 8/9] ui: add move namespace action

[Hannes Laimer <h.laimer@proxmox.com>]

On 2026-03-24 11:56, Dominik Csapak wrote:
> looks mostly fine
> 
> preexisting but slightly confusing:
> 
> the action of a namespace can only be done from that namespace itself,
> not from one level above. maybe it would be sensible to make
> that possible as well?
> 
> otherwise one comment inline
> 
> On 3/19/26 5:13 PM, Hannes Laimer wrote:
>> When moving a namespace, exclude the source namespace and its subtree
>> from the 'New Parent' selector.
>>
>> Signed-off-by: Hannes Laimer <h.laimer@proxmox.com>
>> ---

[..]

>>   @@ -1113,10 +1135,13 @@ Ext.define('PBS.DataStoreContent', {
>>                           if (data.ty === 'group') {
>>                               return Ext.String.format(gettext("Move
>> group '{0}'"), v);
>>                           }
>> -                        return '';
>> +                        return Ext.String.format(gettext("Move
>> namespace '{0}'"), v);
> 
> this tooltip is shown for all entries when hovering, even when he icon
> is not visible, probably due to it never being disabled
> 
> i think verify and remove have the same problem...
> 

yup, they do. can do a follow-up