Skip to content

[Apple Battery Widget] Create new app - #4306

Merged
thyttan merged 6 commits into
espruino:masterfrom
RKBoss6:applebatt
Aug 10, 2026
Merged

[Apple Battery Widget] Create new app#4306
thyttan merged 6 commits into
espruino:masterfrom
RKBoss6:applebatt

Conversation

@RKBoss6

@RKBoss6 RKBoss6 commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Creates a new app based on the style of the iOS battery icon.

Copilot AI review requested due to automatic review settings August 3, 2026 02:22

Copilot AI left a comment

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.

Pull request overview

Adds a new applebatt widget app that displays the current battery percentage in an iOS-style pill indicator.

Changes:

  • Introduces the applebatt widget implementation with charging/low-battery color states.
  • Adds app metadata (id/name/version/storage mapping) and initial ChangeLog entry.

Reviewed changes

Copilot reviewed 3 out of 6 changed files in this pull request and generated 2 comments.

File Description
apps/applebatt/widget.js Implements the widget drawing logic and periodic redraw scheduling.
apps/applebatt/metadata.json Registers the widget with the app loader and defines storage targets/screenshots.
apps/applebatt/ChangeLog Adds initial version entry for the new widget.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread apps/applebatt/widget.js Outdated
Comment thread apps/applebatt/widget.js
@RKBoss6

RKBoss6 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

This looks good to go now!

@thyttan

thyttan commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

If you're making a widget please start the name with wid to make it easy to find!

( @bobrippling I guess we should add this to the PR merge check list? https://github.com/espruino/BangleApps/wiki/App-Contribution )

@thyttan

thyttan commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

To clarify, the appid should start with wid. Not the user facing name.

@RKBoss6

RKBoss6 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

Thanks, just pushed. We could also add it to sanitycheck.js, by checking if type is widget, making sure it has prefix "wid"

@thyttan

thyttan commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

We could also add it to sanitycheck.js, by checking if type is widget, making sure it has prefix "wid"

I think that's a good idea!

@bobrippling

Copy link
Copy Markdown
Collaborator

( @bobrippling I guess we should add this to the PR merge check list? https://github.com/espruino/BangleApps/wiki/App-Contribution )

Good call, how's this?

@RKBoss6

RKBoss6 commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

I think that's a good idea!

Just made a PR for it, #4309

@RKBoss6

RKBoss6 commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

Now that #4309 is merged in, shall we merge this in as well?

@thyttan

thyttan commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

Yes, thanks!

@thyttan
thyttan merged commit ae5b8c1 into espruino:master Aug 10, 2026
2 checks passed
@RKBoss6
RKBoss6 deleted the applebatt branch August 10, 2026 20:14
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.

4 participants