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
New Keyboard UI implementation #445
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Long pressing on a key causes the key to generate a keypress in addition to opening the expanded key menu.
Fixed the long press issue, also added a dim to the keyboard when the popup is on. |
Fixed closing the pop up when clicking on the keyboard padding. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There seem to be a number of hard coded values that should probably be const values but that should be addressed in a follow up issue when code is refactored, possibly when we add non-roman alphabet keyboards?
StringBuilder popupCharacters = new StringBuilder(popupKey.popupCharacters); | ||
switch (popupKey.codes[0]) { | ||
case 110: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
part of the refactor should be replacing these numbers with const values so it is easier to read.
@MortimerGoro Feel free to review the code at your leisure and file any issues as follow ups. |
Fixes #256, Fixes #84, Fixes #333 Added new Keyboard UI
The keyboard code really needs a refactoring, I'll address that in the next keyboard iteration.