Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flexible prompt size is problematic when user has to configure a "deep" option #9

Closed
SigmaBonder opened this issue Jun 19, 2023 · 1 comment · Fixed by #15
Closed
Assignees

Comments

@SigmaBonder
Copy link

SigmaBonder commented Jun 19, 2023

When a user needs to change an existing remote's option that is far in the prompt chain, they need to select "Next" many times in order to get there. In CLI it would be simple, as it just requires pressing Enter to keep the current config value and go to the next one. On a touch screen, the default dialog behavior of adjusting its size to contents invites the user for a whack-a-mole game. It could be better to make the dialog always occupy full screen, even if the empty space may not be visually appealing.

Additionally, some way to go back to the previous prompt would be very useful in this scenario (in case the user happens to click "Next" one time too many).

chenxiaolong added a commit that referenced this issue Jun 20, 2023
This makes it easier to reach the UI elements one-handed and prevents
the dialog action buttons from shifting.

Fixes: #9

Signed-off-by: Andrew Gunnerson <accounts+github@chiller3.com>
@chenxiaolong
Copy link
Owner

chenxiaolong commented Jun 20, 2023

Good point about the whack-a-mole. I've added a new option in #15 that makes all dialog boxes show up at the bottom of the screen. This should keep the reset/cancel/next buttons always in the same place. The appearance of the dialog itself is still exactly the same, except it only grows upwards now, instead of both upwards and downwards.

Additionally, some way to go back to the previous prompt would be very useful in this scenario (in case the user happens to click "Next" one time too many).

This is unfortunately not possible with rclone's interactive configuration API.

@chenxiaolong chenxiaolong self-assigned this Jun 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants