Opened 6 years ago

Last modified 2 years ago

#5835 new defect

getstr() doesn't support the delete key

Reported by: atagar Owned by: atagar
Priority: Low Milestone:
Component: Core Tor/Nyx Version:
Severity: Normal Keywords: curses
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


Curses getstr prompts are dumb, and we need to teach it how to be a sane text editor. Arm's TextInputValidator class [1] supports the left/right arrows, home, end, and esc keys. We should add support there for the delete key too.

This should be easy...

  • add a line here to log keycodes
  • run arm, navigate to the interpretor panel, hit the delete key
  • see in the log what the keycode is
  • add a handler for that keycode to delete the next character

Another minor bug is that hitting backspace when at the start of the line deletes the first character.


Child Tickets

Change History (1)

comment:1 Changed 2 years ago by atagar

Keywords: curses added
Severity: Normal
Summary: Arm getstr prompt doesn't support deletegetstr() doesn't support the delete key
Note: See TracTickets for help on using tickets.