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

Adding touch control to change the display brightness #1

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

Conversation

witnessmenow
Copy link

No description provided.

@jnthas
Copy link
Owner

jnthas commented Jun 26, 2022

Hey! What do you think to turn the use of touch buttons optional through a directive? Trinity provides a facility to use the built-in touch function, but maybe they are not using Trinity, using pin 8 and 9 for other reason or they want other function for touch buttons than set bright. Let me know your opinion.

@witnessmenow
Copy link
Author

Probably makes more sense to make it a boolean config setting in WiFiManager otherwise you technically would need to binaries to enable it. What do you think?

@jnthas
Copy link
Owner

jnthas commented Jun 27, 2022

Makes sense. It's works for me as well. I think the parameter could be something like "Trinity board" and soon we can add more functionalities based on that.

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