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

Command panel clears wrong input on enter #7

Open
ningtan11 opened this issue Nov 11, 2022 · 1 comment
Open

Command panel clears wrong input on enter #7

ningtan11 opened this issue Nov 11, 2022 · 1 comment

Comments

@ningtan11
Copy link
Owner

ningtan11 commented Nov 11, 2022

After typing a long command that might have a minor typo, the entire command is automatically cleared. This means the user has to retype the entire command again, instead of being able to go back and edit the minor typo.

Would be better if the error message is shown but the incorrect input is kept, so the user can easily fix their error.

@soc-pe-bot
Copy link

soc-pe-bot commented Nov 14, 2022

Team's Response

Hi there! Thanks so much for the report.

Clearing the command line after a wrong input (after every input to be more specific) was a feature that we implemented, for various reasons. Firstly, our target users for the app are fast typists, and for such a user group it is critical to maintain a fast typing flow. If there were an incorrect input for instance, it would be faster to simply type the command again, as opposed to either going back with the arrow keys, or using backspace, both of which breaks the flow of typing.

Moreover, Coydir is designed such that the most common functions used in daily operations for a company, are short commands such as view and find. Rarely will a HR executive have to use the add command to individually add (batch-add is much more likely), or the edit command to change all particulars for an employee.

With these considerations in mind, we concluded that it would be much more useful for our target user to have the command line cleared after every input, to provide for an optimal use of the CLI app. Therefore we would like to reject this issue.

Once again, thank you so much for informing us regarding the error.

Cheers,

Coydir

HR is hard, Coydir is easy

(est. 2022)

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: [replace this with your explanation]


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants