eduidl / rtui

TUI tool for ROS
Apache License 2.0
196 stars 9 forks source link

Uppercase shortcut does not working. #3

Closed Teo-Cardoso closed 2 years ago

Teo-Cardoso commented 2 years ago

The shortcut keys are shown as uppercase letters, however, when uppercase is enabled the shortcuts don't work. I see four ways of dealing with this:

  1. Add the uppercase and keep the lowercase letters as well
  2. Remove lowercase shortcuts and add the uppercase letters
  3. Change the letters on the UI to lowercase and keep the shortcuts in lowercase letters
  4. Don't change anything and keep it this way

image

eduidl commented 2 years ago

@Teo-Cardoso Thank you for your feedback! This is the default specification of the TUI library, textual, and I did not intend to make the display and behavior inconsistent. I'm going to choose the third options. I'm fixing this issue in https://github.com/eduidl/rtui/pull/4.

eduidl commented 2 years ago

image