search

jmespath-community / jmespath.terminal

JMESPath exploration tool in the terminal
Apache License 2.0
9 stars 1 forks source link

Fix key binding to toggle output mode #7

Closed kit494way closed 10 months ago

kit494way commented 1 year ago

The help says that output mode can be changed by Ctrl-o. https://github.com/jmespath-community/jmespath.terminal/blob/7c0999d84c22facef099977d54bc08b482a9af78/jpterm.py#L207-L212

Current implementation using Ctrl-p. https://github.com/jmespath-community/jmespath.terminal/blob/7c0999d84c22facef099977d54bc08b482a9af78/jpterm.py#L160-L163

Since it is easier to remember the key bindings for output mode by using o, the first letter of output, I changed the implementation.

kit494way commented 1 year ago

I noticed that the README also states that the key binding is Ctrl-p, so I updated README. I prefer Ctrl-o, this is a breaking change, so please let me know if Ctrl-p is better and I will change it.