-
-
Notifications
You must be signed in to change notification settings - Fork 52
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
Added writing query and content to clipboard #46
Conversation
Cargo.toml
Outdated
jaq-core = "1.2.1" | ||
jaq-interpret = "1.2.1" | ||
jaq-parse = "1.0.2" | ||
jaq-std = "1.2.1" | ||
promkit = "0.4.3" | ||
promkit = { path = "/Users/antonsuprun/Projects/promkit/promkit" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note: will remove this once it's ready to merge
0553f45
to
fa555d1
Compare
Thanks again for your suggestion on the promkit change; let me know if you have any other suggestions :) (Once again; apologies for the delay on this 🙏 ) |
@anton-suprun LGTM! But please merge until promkit release (and edit the version after confirming release) 🙇 |
Thank you 😃 ! And yes, acknowledged. Will do that ! |
@anton-suprun And also, please add the items about new key-bindings to https://github.com/ynqa/jnv/blob/main/README.md?plain=1#L96-L118. |
@anton-suprun I just released promkit as v0.4.5 🎉 |
Awesome!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! @anton-suprun Thanks for your contribution~ 👍
Description
Addresses issue: Feature Request: Copy current JSON displayed #27
Added two methods to Jnv
content_to_clipboard
- writes currently visible json to clipboard and adds a hint that content was copied to clipboardquery_to_clipboard
- writes current jq query to clipboard and adds a hint that the query was copied to clipboardAdded two new shortcuts:
ctrl-o
- copies content to clipboardctrl-q
- copies query to clipboardScreencasting
Note: I have a clipboard manager (via Raycast); that's the pop-up you're seeing; I use it to show the latest copied text
Screen.Recording.2024-07-24.at.11.18.54.PM.mov
Notes
Please feel free to request any changes to code structure, shortcuts, etc. I am happy to receive any of your feedback on this 😊
Dependency promkit PR
Resolves #27