pudb icon indicating copy to clipboard operation
pudb copied to clipboard

Command history with cursor up in internal shell

Open ulope opened this issue 5 years ago • 8 comments

When inside the internal shell the cursor-up key just selects the lines in history with no (apparent) useful function.

It would be nice if it could instead access the history (as does ctrl-p/n).

ulope avatar Jul 05 '19 17:07 ulope

While I understand the request, I fear that that would be inconsistent with how navigation in Urwid works. Under your suggested change, Ctrl-X would likely be the only way to navigate away from the (internal) command line.

inducer avatar Jul 06 '19 17:07 inducer

I find this annoying too. You can navigate history in the internal shell with Ctrl-p / Ctrl-n. Going through the history is useful, especially for long outputs like tracebacks. But I would prefer if the arrow keys defaulted to history.

asmeurer avatar Jul 09 '19 02:07 asmeurer

Hmm, OK. I cold be convinced, if one of you were to produce a PR.

inducer avatar Jul 09 '19 21:07 inducer

Agree, this is super annoying. It's really messing up my muscle memory.

erik-megarad avatar Aug 08 '19 00:08 erik-megarad

Perhaps this could be a configurable preference?

up/down going back/forward through shell history is fundamentally hardwired into my expectations of how REPLs work. I'm loving PUDB but this is breaking my brain.

What would be involved in putting together a PR for this?

ned2 avatar May 05 '20 03:05 ned2

IMO it should just be changed. Should be a simple PR.

asmeurer avatar May 05 '20 05:05 asmeurer

Given the overwhelming sentiment on this, I'm on board.

inducer avatar May 05 '20 14:05 inducer

I've created submitted a PR for this at #392. I figured leaving the existing ctrl p/n bindings made sense?

ned2 avatar Jun 28 '20 01:06 ned2