[pbs-devel] applied: [PATCH proxmox-backup 3/5] ui: datastore content: add context menu to groups and snapshots

Thomas Lamprecht t.lamprecht at proxmox.com
Tue Nov 28 16:27:42 CET 2023


Am 28/11/2023 um 10:44 schrieb Dominik Csapak:
> and show the relevant actions. They will be forwarded to the controller,
> so we can reuse that code without big refactoring them into another
> class/place.
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
> sadly no keyboard navigation in there.. but i'm looking into it
> 
>  www/datastore/Content.js | 114 +++++++++++++++++++++++++++++++++++++++
>  1 file changed, 114 insertions(+)
> 
>

applied this one for now, thanks!

Albeit, I'm wondering if we should add the ID into the menu header, for
snapshots that tends to be a bit large though...

Oh, and fwiw, we could add a "Copy Text" for the cells where it makes
sense (ID of group or snapshot, comment firt line, maybe date) so that
one has it slightly easier if they found the wanted snapshot/group here
and what to use that ID for, e.g., finding that in the PVE storage
content UI, but no hard feelings, just as an idea.




More information about the pbs-devel mailing list