-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
add support of configurable extra keys #650
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
Conversation
|
Merged with PR #605 that would be a great way to make the overlay customisable ! As long as the name of control keys are obvious and ascii, like LEFT, TAB, "-" etc. Or Unicode if the user wants to add "é" or greek letters or whatever he wants. |
|
And the strings don't have to be of length 1, one could add keywords like |
|
@robertvandeneynde Sorry for the delay here! Now that #605 has been merged, could you look at rebasing your changes on top of that? |
|
I'm WAITING FOR merging this PR!! |
|
@fornwall, that's not my PR ! However I think I have a good view of how a good customisation would be (as me and others talked on another PR and issue) but I don't know how to have a local build in order to code the change myself. |
|
Waiting for this merge, because I do not want to downgrade to version 0.61 |
|
@neverwin could you rebase your changes now that #605 has been merged ? About the choice of syntax, we have extrakeys must therefore be an list of list, what happens if is is not ? @fornwall I suggest at least those options (and documented somewhere on the wiki) :
I think this should be implemented in this PR. More ideas for future enhancement/PR include :
I could code it myseslf and create my own PR if I knew how to build the app. |
|
Closing, since #788 incorporated these changes |
|
Sorry for the late reply due to some personal difficulties. Thanks for your efforts about this function 👍. |
People can change extra keys to meet their needs. For example, add one line in termux.properties:
That's the default one of Termux.