Skip to content

Add low battery warning display and handling - #1660

Open
pmsobrado wants to merge 1 commit into
spruceUI:Developmentfrom
pmsobrado:fix_low_power_warning_black_screen
Open

pmsobrado wants to merge 1 commit into
spruceUI:Developmentfrom
pmsobrado:fix_low_power_warning_black_screen

Conversation

@pmsobrado

Copy link
Copy Markdown
Contributor

Here's my attempt to fix low battery warning making the screen black when the user accepts the message, at least on Flip devices. I tested this myself on my Flip with Spruce 4.3.4 running.

Also, I made pressing A to dismiss not perform an A press action on whatever screen behing the warning message.

Maybe the code can be improved, cleaned, or maybe I touched the wrong files. All I can say is that it works for me. Feel free to change the code or close the PR and make your own fix :)

display -t "Battery has $CAPACITY% left. Charge or shutdown your device." \
--add-image "/mnt/SDCARD/spruce/imgs/displayAcknowledge.png" 1.0 240 middle

/mnt/SDCARD/spruce/flip/bin/python3 - <<'PY'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DEVICE_PYTHON3_PATH is what should be used

--add-image "/mnt/SDCARD/spruce/imgs/displayAcknowledge.png" 1.0 240 middle

/mnt/SDCARD/spruce/flip/bin/python3 - <<'PY'
import os

@chrisj951 chrisj951 Sep 15, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is also very device specific based on inputs being read / values checked. You should be able to combine display and confirm() or acknowledge() from helper functions to get what you want and not have it be device specific. Anything broken there should be fixed vs spinning off a flip specific low battery warning

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants