-
Notifications
You must be signed in to change notification settings - Fork 54
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
Enhancements to tiny OLEDs, OTA beautified #155
base: main
Are you sure you want to change the base?
Conversation
Date: Mon Sep 30 06:59:16 2024 +0900 new file: .gitignore modified: README.md modified: data/defaultprefs.txt modified: doc/example config.txt modified: src/main.cpp modified: tools/Esp32_radio_init.ino
modified: src/main.cpp
Out of curiosity, why not merge this? I've merged your most recent changes... there shouldn't be any conflict. |
@Edzelf I am upvoting this PR, updates for the small OLED are very welcome. |
@aly-fly you can try it out on my fork if you like. |
Will do and report back. Hint: It might be easier for the author to test and merge smaller updates, like separate OLED and separate OTA and not everything together. Just a guess. I know I would rater see it that way as an owner of a such popular repository. |
Yeah, I got a bit carried away... although part of the problem with that was... I actually worked on both bits of code simultaneously. There was no separating them once I had begun... and they do interact with each other... a bit. |
A quick summary of what's in this (as included in the main.cpp):
OLED 64x128 improvements: blank screen with backlight time (both available in prefs), character limit for small displays. Also: OTA beautified, fixed rotary timeout display, new esp-idf ADC (the new number is 12 for attenuation instead of 11).
I added a bunch of functions to help with small displays but I think they only apply to the tiny OLEDS. But they can be applied to other displays should the need arise.
I made a big change to the OTA too. It looks pretty good on my Tiny OLED. It didn't display any update information before at all. It was kind of a PITA to get it working the way it's supposed to.