Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(lwjgl3): fix and split char and raw key handler #12

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

DarkWeird
Copy link
Contributor

###Contains

Splitting key and char event handler.
This needs after LWJGL3 introduce. It handle raw keys and text input separately.
Fixes #11

How To Test

  1. create world with ModularComputers
  2. Place Monitor and setup modular computer ( idk case, you must works with console or/and program editing)
  3. Open console and try input:
    1. any english chars (seems, en Layout must be enabled)
    2. Try DELETE, BACKSPACE, LEFT, RIGHT, HOME,END, ENTER, UP, DOWN keys (P.S. UP -DOWN works with history commands)
  4. Open program editing(terminal) and try input:
    1. any english chars (seems, en Layout must be enabled)
    2. DELETE and BACKSPACE Keys
    3. UP, DOWN, LEFT,RIGHT Keys
    4. HOME, END Keys
    5. ENTER key
    6. Ctrl+UP, Ctrl+DOWN (right and left Ctrl keys) - text scale changing.
    7. Ctrl+S - saving
    8. Ctrl+X - exit - appears question
      8.1. try any chars except Y and N, terminal must not receive any char.
      8.2. type Y or N. - Must be KeyModifier independent(works with any Ctrl/shift/alt key pressed)
    9. Ctrl+E - display error (seems must be used after script compilation failed)
    10. Ctrl+G - GoTo line - there you must type digit for go to line.
      10.1. typing any chars except digits, terminal must not receive any char.
      10.2 type digits must appears.
      10.3 press enter must invoke goto line with typed digits before (not sure)
    11. Ctrl+V - Paste from clipboard

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.

Proper fix for #10 - better key/char handling for ComputerTerminalWidget
1 participant