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

Add Options menu and Combo Configuration Menu #1

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

Conversation

diwo
Copy link

@diwo diwo commented Feb 18, 2020

This is a preview of the combo config UI. It cannot compile against libtesla@master since the lib is currently missing APIs to allow Gui to capture keysHeld and to update combo keys. This PR also does not include changes to existing UI elements to access this new UI.

The combo config UI asks the user to hold down the new key combo for 2.5 seconds. The user then has to release all the keys, and then input the new combo again to confirm the change. The user can leave the UI without changing the combo by pressing KEY_PLUS twice.

Screenshots:
https://imgur.com/yZ12kx6
https://imgur.com/X22zqBL
https://imgur.com/BNd9JuL

@diwo diwo changed the title (Preview) Overlay combo keys configuration UI Add Options menu and Combo Configuration Menu Mar 6, 2020
@diwo
Copy link
Author

diwo commented Mar 6, 2020

@WerWolv this one is ready now, depends on WerWolv/libtesla#23

@diwo
Copy link
Author

diwo commented Mar 6, 2020

Note that there's no safeguard against assigning stupid key combos like A button by itself. In case someone does go through with it, they may have to take their SD card out of their switch and delete the tesla config file on a computer.

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.

1 participant