-
Notifications
You must be signed in to change notification settings - Fork 118
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
Keybindings? #117
Comments
Hi👋 thanks for creating your first issue for IOSF, Will get back to you soon !. |
Hey @pranavbaburaj . I think this would be a awesome features to work and this will also increase the UX as such in the cli. We can start working on it to make this happen |
@pranavbaburaj Are we working on this or do we need any discussion on it? |
If you are busy, I can also pick it up |
ye, I have exams. I will work on it after the exams. 🙂 |
@GouravSardana, I am a gssoc participant. Can I work on this issue? |
Yeah sure. |
Go ahead and work on it. In your PR tag this issue |
@GouravSardana @pranavbaburaj I noticed that when we are searching in dynamic-cli we can use arrow keys to navigate the results that are shown as a list. Can you tell me in which file it is implemented? |
@GouravSardana Sir, also I would like to ask if it would be possible to have a small call. I have some doubts that I wanted to clarify. |
Those keybindings are parts of the tui library we are using, not a part of out project repo. |
Understood @pranavbaburaj sir, thanks! |
@GouravSardana I would like to work on this issue so could you please assign this issue to me. |
🚀 Feature
My idea:
After displaying the results(from StackExchange), we can open up the stdin to receive keypresses. So that we can perform specific tasks(or functions) with keyboard shortcuts.
Examples:
ctrl+s
- > to save results to a filectrl+r
-> Reload the results.And many more...
The
keyboard
module will help us listen for keypresses.Have you read the Contributing Guidelines on Pull Requests?
Yes
Motivation
I implemented this feature while I was working on a project. I thought we could implement this feature in dynamic-cli because it makes it easier for people to use the cli.
Pitch
explained above
The text was updated successfully, but these errors were encountered: