BangleApps icon indicating copy to clipboard operation
BangleApps copied to clipboard

New apps: bmoface, doomguy, and meseeks!

Open LucienStorm opened this issue 2 months ago • 9 comments

LucienStorm avatar Oct 20 '25 23:10 LucienStorm

Never stop improving :)

LucienStorm avatar Oct 20 '25 23:10 LucienStorm

These look really nice :)

RKBoss6 avatar Oct 21 '25 01:10 RKBoss6

Thanks for the apps! They appear to all be the same code but a different app for different images, are you able to combine them into one app? Perhaps that the user can select between when they install?

bobrippling avatar Oct 21 '25 15:10 bobrippling

I suppose I could, however these are three completely unrelated character themes, with different offerings on the clock info placement and interactivity. They were all made using the Advanced Casio face as a template, and I left them that way to show people how easy it can be to make a new one.

If you just want to switch between the faces/clocks, there are a couple apps that allow you to either schedule clocks, or cycle through them with a tap.

LucienStorm avatar Oct 21 '25 17:10 LucienStorm

Fair enough - I'll merge (once the lint is ok) but please bear in mind there's been 5 PRs, which I've read through all as they go, and each app is ~700 lines of code, so combined this takes up a lot of maintainer free-time

bobrippling avatar Oct 22 '25 08:10 bobrippling

I notice stardateclock_wbin has appeared - did you mean to add that too?

bobrippling avatar Oct 30 '25 13:10 bobrippling

facepalm No, I didn't mean to add that one yet.

Would it be best at this time to cancel this one and retry with the 3 separate pull requests?

Lucien Storm

On October 30, 2025 8:08:21 AM CDT, Rob Pilling @.***> wrote:

bobrippling left a comment (espruino/BangleApps#4041)

I notice stardateclock_wbin has appeared - did you mean to add that too?

-- Reply to this email directly or view it on GitHub: https://github.com/espruino/BangleApps/pull/4041#issuecomment-3467906819 You are receiving this because you authored the thread.

Message ID: @.***> Through K-9 Mail on Android

LucienStorm avatar Oct 30 '25 19:10 LucienStorm

Nah that's alright - I can tinker with the commits and just do the three if you're ok with that?

bobrippling avatar Oct 30 '25 22:10 bobrippling

I'm not sure if I responded to this, I'm ok with that. Thank you for your help.

Lucien Storm

On October 30, 2025 5:29:32 PM CDT, Rob Pilling @.***> wrote:

bobrippling left a comment (espruino/BangleApps#4041)

Nah that's alright - I can tinker with the commits and just do the three if you're ok with that?

-- Reply to this email directly or view it on GitHub: https://github.com/espruino/BangleApps/pull/4041#issuecomment-3470483293 You are receiving this because you authored the thread.

Message ID: @.***> Through K-9 Mail on Android

LucienStorm avatar Nov 13 '25 12:11 LucienStorm

Sorted :) will merge when the checks pass

bobrippling avatar Nov 25 '25 08:11 bobrippling

Looks like there's the lint firing, same as before if you get change to sort them:

/home/runner/work/BangleApps/BangleApps/apps/bmoface/app.js
Warning:    1:7   warning  'storage' is assigned a value but never used    no-unused-vars
Warning:   16:10  warning  'getBackgroundImage' is defined but never used  no-unused-vars
/home/runner/work/BangleApps/BangleApps/apps/bmoface/settings.js
Warning:   47:4  warning  'back' is not defined  no-undef
/home/runner/work/BangleApps/BangleApps/apps/doomguy/app.js
Warning:   16:10  warning  'resetHitCounter' is defined but never used  no-unused-vars
/home/runner/work/BangleApps/BangleApps/apps/doomguy/doomguy.settings.js
Warning:   66:4  warning  'back' is not defined  no-undef
/home/runner/work/BangleApps/BangleApps/apps/meseeks/app.js
Warning:   1:7  warning  'storage' is assigned a value but never used  no-unused-vars

bobrippling avatar Nov 25 '25 08:11 bobrippling