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

Replace forms with a TUI #225

Closed
daonb opened this issue Jun 5, 2022 · 1 comment · Fixed by #249
Closed

Replace forms with a TUI #225

daonb opened this issue Jun 5, 2022 · 1 comment · Fixed by #249
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone

Comments

@daonb
Copy link
Contributor

daonb commented Jun 5, 2022

What's missing? Please describe your flow and where this feature will help.
The forms T7 uses today are web forms and seem out of place.
This issue is about refactoring the forms to use Terminal user Interface.

Instead of a form, t7, should open a small terminal window and let the user
key in their values. TUI also means there's not longer a submit button:
the instance the user entered the last value the form is submitted.

the modal will just have a large 'X' at the bottom

Additional context
gh CLI is a fine example of TUI forms and T7 should follow its lead.

@daonb daonb added the enhancement New feature or request label Jun 5, 2022
@daonb daonb added this to the 1.0.0 milestone Jun 5, 2022
@daonb daonb added the good first issue Good for newcomers label Jun 12, 2022
@daonb daonb changed the title Replace forms with TUI Replace forms with a TUI Jun 12, 2022
@daonb
Copy link
Contributor Author

daonb commented Jul 3, 2022

The component should accept an array of maps where each map has:

  • description
  • default
  • validValues - optional, e.g. ["yes", "no"]

After the user entered all the fields (or clicked enter to use a default) the component will finish with an array of values.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant