From f243c0972b9426b0fa25e46b665a4c2d80479754 Mon Sep 17 00:00:00 2001 From: stephenPspackman <93166870+stephenPspackman@users.noreply.github.com> Date: Mon, 13 Dec 2021 10:33:58 -0800 Subject: [PATCH 1/6] Update ChangeLog --- apps/pooqroman/ChangeLog | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/apps/pooqroman/ChangeLog b/apps/pooqroman/ChangeLog index 9debf0efe..c4f3171d3 100644 --- a/apps/pooqroman/ChangeLog +++ b/apps/pooqroman/ChangeLog @@ -1,2 +1,3 @@ -0.01: New App! -0.02: Make internal menu time out + small fixes +0.01: Initial check-in. +0.02: Make internal menu time out + small fixes. +0.03: Autolight feature. From fcce9b7336fb617a8a6888197c2c15e92e884693 Mon Sep 17 00:00:00 2001 From: stephenPspackman <93166870+stephenPspackman@users.noreply.github.com> Date: Mon, 13 Dec 2021 10:35:21 -0800 Subject: [PATCH 2/6] Update apps.json --- apps.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps.json b/apps.json index 0826bbf81..4a7d02093 100644 --- a/apps.json +++ b/apps.json @@ -4734,7 +4734,7 @@ { "id": "pooqroman", "name": "pooq Roman watch face", "shortName":"pooq Roman", - "version":"0.02", + "version":"0.03", "description": "A classic watch face with a certain dynamicity. Most amusing in 24h mode. Slide up to show more hands, down for less(!). By design does not support standard widgets, sorry!", "icon": "app.png", "type": "clock", From 186258e3984a1a2f2586cbf576e8b99dcd779e83 Mon Sep 17 00:00:00 2001 From: stephenPspackman <93166870+stephenPspackman@users.noreply.github.com> Date: Mon, 13 Dec 2021 11:17:24 -0800 Subject: [PATCH 3/6] Update app.js Add an option to illuminate the display on twist. --- apps/pooqroman/app.js | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/apps/pooqroman/app.js b/apps/pooqroman/app.js index d59d4ef6c..52fffa76f 100644 --- a/apps/pooqroman/app.js +++ b/apps/pooqroman/app.js @@ -138,6 +138,10 @@ class RomanOptions extends Options { onchange: x => this.calendric = x, format: x => ['none', 'day', 'date'][x] }, + 'Auto-Illum.': { + init: _ => this.autolight, + onchange: x => this.autolight = x + }, Defaults: _ => {this.reset(); this.interact();} }; } @@ -164,6 +168,7 @@ RomanOptions.defaults = { alarmFg: '#f00', timerFg: '#0f0', activeFg: g.theme.fg2, + autolight: true, }; ////////////////////////////////////////////////////////////////////////////// @@ -664,8 +669,8 @@ class Clock { this.listeners = { lcdPower: on => on ? this.active() : this.inactive(), - charging: () => {face.doIcons('charging'); this.active();}, - lock: () => {face.doIcons('locked'); this.active();}, + charging: _ => {face.doIcons('charging'); this.active();}, + lock: _ => {face.doIcons('locked'); this.active();}, faceUp: up => {this.conservative = !up; this.active();}, drag: e => { if (this.t0) { @@ -694,6 +699,8 @@ class Clock { } } }; + this.options.autolight && + (this.listeners.twist = _ => Bangle.setLCDBrightness(1)); } redraw(rate) { @@ -728,7 +735,6 @@ class Clock { } const delay = rate - now % rate + 1; this.refresh = true; - if (rate !== prev) { this.inactive(); this.redraw(rate); From 134935dc469f61fbe6be5053e60a97ce87f9e565 Mon Sep 17 00:00:00 2001 From: stephenPspackman <93166870+stephenPspackman@users.noreply.github.com> Date: Mon, 13 Dec 2021 11:52:10 -0800 Subject: [PATCH 4/6] Update app.js Debugged. Also, it seems that on Bangle 2, the responsible way to do this is with LCDPower, not LCDBrightness. --- apps/pooqroman/app.js | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/apps/pooqroman/app.js b/apps/pooqroman/app.js index 52fffa76f..9e13e6e03 100644 --- a/apps/pooqroman/app.js +++ b/apps/pooqroman/app.js @@ -668,10 +668,10 @@ class Clock { this.options.on('done', () => this.start()); this.listeners = { - lcdPower: on => on ? this.active() : this.inactive(), charging: _ => {face.doIcons('charging'); this.active();}, lock: _ => {face.doIcons('locked'); this.active();}, faceUp: up => {this.conservative = !up; this.active();}, + twist: _ => this.options.autolight && Bangle.setLCDPower(true), drag: e => { if (this.t0) { if (e.b) { @@ -699,8 +699,6 @@ class Clock { } } }; - this.options.autolight && - (this.listeners.twist = _ => Bangle.setLCDBrightness(1)); } redraw(rate) { @@ -771,7 +769,7 @@ class Clock { this.rates.clock = this.timescales[this.options.resolution]; this.active(); for (const l in this.listeners) { - Bangle.on(l, this.listeners[l]); + this.listeners[l] && Bangle.on(l, this.listeners[l]); } Bangle.setUI('clock'); return this; From abfb7fc8a47a92648a23a21e996a2d62423fe062 Mon Sep 17 00:00:00 2001 From: stephenPspackman <93166870+stephenPspackman@users.noreply.github.com> Date: Mon, 13 Dec 2021 12:51:01 -0800 Subject: [PATCH 5/6] Update README.md FIx a typo, add a couple of sentences about autolight. --- apps/pooqroman/README.md | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/apps/pooqroman/README.md b/apps/pooqroman/README.md index b41a4a316..87acea9ca 100644 --- a/apps/pooqroman/README.md +++ b/apps/pooqroman/README.md @@ -13,9 +13,12 @@ you can alter the number of hands on the display. When the watch is unlocked, sl There's also a setting that displays the second hand, but only if the watch is perfectly face-to-the-sky, in case you want the ability to check the _exact_ time, hands free, without the impact on battery life this usually entails. -Although we genrally obey the system-wide theming, you can long press on the display for a menu of additional options specific to the face. +Although we generally obey the system-wide theming, you can long press on the display for a menu of additional options specific to the face. You can also override the system 12/24 hour setting just for this face here, since it's, well, a rather different experience than with numeric displays. +By default, there is a backlight that comes on when you twist your wrist. This, of course, somewhat increases power draw and could be +annoying in an intentionally dark environment, so there is an option to disable it. + One other thing: there's some integration with system timers and alarms; they will show as small pips at the appropriate places in the day around the display. When they come within an hour, the pips turn to crosses relating to the minute hand, and the minute hand turns itself on. When timers are mere seconds away, the display changes again and the second hand activates itself, so you From c1cbc3937779482f43cd214263ebcd5d1b748517 Mon Sep 17 00:00:00 2001 From: stephenPspackman <93166870+stephenPspackman@users.noreply.github.com> Date: Mon, 13 Dec 2021 13:00:29 -0800 Subject: [PATCH 6/6] Update app.js Remove an unneeded piece of defensiveness. --- apps/pooqroman/app.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/pooqroman/app.js b/apps/pooqroman/app.js index 9e13e6e03..bed8ef3d2 100644 --- a/apps/pooqroman/app.js +++ b/apps/pooqroman/app.js @@ -769,7 +769,7 @@ class Clock { this.rates.clock = this.timescales[this.options.resolution]; this.active(); for (const l in this.listeners) { - this.listeners[l] && Bangle.on(l, this.listeners[l]); + Bangle.on(l, this.listeners[l]); } Bangle.setUI('clock'); return this;