Skip to content

Conversation

@cwh42
Copy link
Contributor

@cwh42 cwh42 commented Nov 16, 2016

hotkey-help-ncurses
It was not possible to use Shift-F1 as an universal hotkey-helper-hotkey in ncurses. It depends on the terminal whether it reaches the application, or not.
So I added two hotkeys:

  • Shift-F1, which only works on a linux console (seen by the code as F13)
  • Ctrl-D F1, working even in other terminals, like xterm.

I chose Ctrl-D as a prefix, because most of the other advanced hotkeys also do so.

@cwh42 cwh42 force-pushed the master branch 2 times, most recently from 0c66741 to dd63c5b Compare November 21, 2016 12:23
src/NCDialog.cc Outdated

break;

case KEY_F( 13 ):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a comment that says that this is the equivalent to Shift-F1 for certain $TERM types (linux? xterm?).

@shundhammer
Copy link
Contributor

Besides the missing comment above, LGTM.

@cwh42
Copy link
Contributor Author

cwh42 commented Nov 21, 2016

Comment is there, now.

@shundhammer
Copy link
Contributor

LGTM

@cwh42 cwh42 merged commit d456f20 into libyui:master Nov 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants