make notifications frame indicators bounce on text message #715
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
crossan007
force-pushed
the
adjust_frame_notifications
branch
2 times, most recently
from
d3b75b2 to
b0d1793
Compare
|
cool! can you add an include for - see the failing nrf52 build. on the esp32 targets it worked okay because I assume something in esp32 arduino land is implicitly including it? |
crossan007
force-pushed
the
adjust_frame_notifications
branch
from
d39398a to
391c73a
Compare
mostly working ui panel notifications / ack adjust pinned dep
If there is an active notification, go to it If there are no notifications, go to the first frame update pinned esp8266-oled-ssd1306
crossan007
force-pushed
the
adjust_frame_notifications
branch
from
b7034c4 to
40985db
Compare
|
Hi @crossan007 You are certainly welcome back anytime (and your home automation stuff looks cool) but just to keep the PR queue tidy, I'm closing this one for now. |
|
Heh ok. I had this mostly working, but the API wasn't quite tidy |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update the UI frame symbols on the bottom of the OLED to "bounce" if that frame has a notification (currently only works with the text messages).
Double-pressing the user button takes you to the first screen with active notifications.
Opening a screen with an active notification "acknowledges" the notification, and causes bouncing to stop.
bounce.mp4
Double-pressing the user button when there are no active notifications takes you to the first screen
Depends on: meshtastic/esp8266-oled-ssd1306#3