bruno icon indicating copy to clipboard operation
bruno copied to clipboard

Comment with keybinding (cmd + /)

Open gabrielbidula opened this issue 1 year ago • 6 comments

In a JSON payload interface, similar to other API GUIs, it would be beneficial to support code commenting within the payload. This feature should enable users to exclude specific parts of the request through comments. Additionally, implementing a keyboard shortcut for commenting, such as 'Cmd + /' on macOS, would enhance usability. This shortcut should facilitate both single-line and multi-line comments, allowing users to comment out a line, move to the next line, and repeat the process as desired. I haven't found any open pull requests addressing this, so I'm providing this suggestion. Thank you for your efforts in developing the application.

Cheers

gabrielbidula avatar Jan 25 '24 17:01 gabrielbidula

That would be a very valuable feature! It's indeed something I miss from Postman.

Drowze avatar Apr 23 '24 22:04 Drowze

Can I contribute to this feature?

samhoooo avatar Jul 12 '24 15:07 samhoooo

Hi @sanjai0py, I implemented this feature in PR https://github.com/usebruno/bruno/pull/2634

I am a begineer and would like to get some suggestions. Thank you!

samhoooo avatar Jul 14 '24 00:07 samhoooo

I was just looking for this feature. I hope it gets merged in soon!

etler avatar Jul 14 '24 17:07 etler

Or like in VS Code "Ctrl + K + C" to comment ("Ctrl + K + U" to uncomment)

Ctrl + K + U -> capitalizes at the moment Alt Gr + 7 -> puts inside { } alt Gr + 8 -> puts inside [ ]

eifcgn24 avatar Sep 05 '24 12:09 eifcgn24

Is there a reason this is still open? The PR was merged quite a while ago.

AshkanArabim avatar Oct 02 '24 03:10 AshkanArabim

yep, it's missing the keybinding for windows & linux users. Ctrl+/

LaxmanChoudhary avatar Oct 09 '24 14:10 LaxmanChoudhary

this should close it. Let's see. 🙄

LaxmanChoudhary avatar Oct 09 '24 17:10 LaxmanChoudhary

nice catch @LaxmanChoudhary. Can we close this issue after @LaxmanChoudhary 's PR is merged?

samhoooo avatar Oct 29 '24 14:10 samhoooo

I can see that we recently switched to the CodeMirror's built-in command toggleComment which apparently doesn't enable it for json file mode.

ramki-bruno avatar Jan 15 '25 10:01 ramki-bruno

There is jsonc, but not sure if CodeMirror supports. Will check and get back on this.

ramki-bruno avatar Jan 15 '25 10:01 ramki-bruno