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.csa...@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! _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
