[pve-devel] applied: [PATCH] tui: focus next button by default
Thomas Lamprecht
t.lamprecht at proxmox.com
Thu Jun 22 14:54:32 CEST 2023
Am 21/06/2023 um 16:48 schrieb Dominik Csapak:
> except the password dialog, since the user must provide input
>
> to do that, we have to set the focus index on all relevant views
>
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>
> not sure if this is the correct approach, also the extra parameter feels
> slightly wrong, but didn't found a nicer way to do this
>
> any errors from focusing will be ignrored, but that shouldn't happen
> anyway until we add/remove buttons and the index changes
>
> alternatively we could create a second 'new_with_focus_next' (or
> 'without') that gets called respectively, but also seems a bit weird for
> that
>
> proxmox-tui-installer/src/main.rs | 101 +++++++++++++++---------------
> 1 file changed, 52 insertions(+), 49 deletions(-)
>
>
seemingly forgot to write: I applied this one, but avoided setting the focus
on the install button for the summary step, as that can be a bit unexpected,
or even dangerous, thanks!
More information about the pve-devel
mailing list