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

Simplify keyboard shortcuts #20

Closed
jsumners-nr opened this issue Mar 26, 2025 · 0 comments · Fixed by #21
Closed

Simplify keyboard shortcuts #20

jsumners-nr opened this issue Mar 26, 2025 · 0 comments · Fixed by #21

Comments

@jsumners-nr
Copy link
Contributor

At the moment, in-app keyboard shortcuts require using the ctrl key. For example, in order to quit the application you have to press ctrl+q. This isn't typical of a TUI, e.g. see how ncdu does things.

The reason we need the composite shortcuts is due to the various input fields within the application. For example: the search box. When searching, if the query term(s) contains a shortcut key, e.g. q, the shortcut will be invoked when that key is pressed in order to add it to the query. Thus, in order to solve this issue, we need to separate recognizing application navigation from field inputs.

@newrelic-node-agent-team newrelic-node-agent-team moved this to Triage Needed: Unprioritized Features in Node.js Engineering Board Mar 26, 2025
@github-project-automation github-project-automation bot moved this from Triage Needed: Unprioritized Features to Done: Issues recently completed in Node.js Engineering Board Apr 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done: Issues recently completed
Development

Successfully merging a pull request may close this issue.

1 participant