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

User experience improvement - app battery icon #3979

Merged
merged 4 commits into from
May 28, 2024
Merged

User experience improvement - app battery icon #3979

merged 4 commits into from
May 28, 2024

Conversation

neilhao
Copy link
Contributor

@neilhao neilhao commented May 27, 2024

For Station G2 or any board that does not have the BATTERY_PIN function defined, the current firmware can accurately determine that the board does not have a battery installed.

However, the current firmware will send batteryChargePercent = 0 to the APP for boards without battery, and the APP will display a battery empty icon. Users hope that boards without BATTERY_PIN will display a plug icon in the APP.

Ps. This is a PR about user experience and does not involve any bug fixes.

src/PowerStatus.h Outdated Show resolved Hide resolved
@thebentern thebentern merged commit 038413f into meshtastic:master May 28, 2024
81 checks passed
kimgnguyen added a commit to kimgnguyen/meshtastic that referenced this pull request May 29, 2024
commit af9d825
Author: Ben Meadors <[email protected]>
Date:   Tue May 28 19:25:19 2024 -0500

    Send own node-info earlier and move others to the end of want-config flow (meshtastic#3949)

    * Send own node-info earlier and move others to the end of want-config flow

    * Special nonce skips other nodeinfos

    * Missed it

commit 038413f
Author: Neil Hao <[email protected]>
Date:   Tue May 28 19:30:15 2024 +0800

    User experience improvement - app battery icon (meshtastic#3979)

    * 'app_battery_icon'

    * Undo VS automatic modifications to this file

    * 'app_battery_icon_2'
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