battery percentage

Firmware/components/backend-esp/include/cardboy/backend/esp/bat_mon.hpp

@@ -18,6 +18,7 @@ public:
     float          get_voltage() const;
     float          get_charge() const;
     float          get_current() const;
+    float          get_percentage() const;
 
     void pooler(); // FIXME:
 private:
@@ -33,6 +34,7 @@ private:
     volatile float _voltage;
     volatile float _current;
     volatile float _charge;
+    volatile float _percentage;
 
     TaskHandle_t _pooler_task;
 };

Firmware/components/backend-esp/src/bat_mon.cpp

@@ -48,6 +48,8 @@ static constexpr uint16_t DesignCapMah = 180; // 100mOhm
 
 constexpr float mahToCap(float mah) { return mah * (1000.0 / 5.0) * RSense; }
 constexpr float capToMah(uint16_t cap) { return cap * (5.0 / 1000.0) / RSense; }
+// lsb is 1/256%
+constexpr float regToPercent(uint16_t reg) { return static_cast<float>(reg) / 256.0f; }
 constexpr float regToCurrent(uint16_t reg) {
     return static_cast<float>(static_cast<int16_t>(reg)) * 0.0015625f / RSense; // Convert to mA
 }
@@ -103,6 +105,7 @@ void BatMon::pooler() {
         _charge  = capToMah(ReadRegister(0x05));
         _current = regToCurrent(ReadRegister(0x0B));
         _voltage = regToVoltage(ReadRegister(0x09));
+        _percentage = regToPercent(ReadRegister(0x06));
         vTaskDelay(pdMS_TO_TICKS(10000));
         if (_voltage < 3.0f) {
             Shutdowner::get().shutdown();
@@ -113,3 +116,4 @@ void BatMon::pooler() {
 float BatMon::get_voltage() const { return _voltage; }
 float BatMon::get_charge() const { return _charge; }
 float BatMon::get_current() const { return _current; }
+float BatMon::get_percentage() const { return _percentage; }

Firmware/components/backend-esp/src/esp_backend.cpp

@@ -78,6 +78,7 @@ public:
     [[nodiscard]] float voltage() const override { return BatMon::get().get_voltage(); }
     [[nodiscard]] float charge() const override { return BatMon::get().get_charge(); }
     [[nodiscard]] float current() const override { return BatMon::get().get_current(); }
+    [[nodiscard]] float percentage() const override { return BatMon::get().get_percentage(); }
 };
 
 class EspRuntime::StorageService final : public cardboy::sdk::IStorage {

Firmware/sdk/apps/lockscreen/src/lockscreen_app.cpp

@@ -7,6 +7,7 @@
 
 #include <algorithm>
 #include <array>
+#include <cmath>
 #include <cstdio>
 #include <ctime>
 #include <string>
@@ -31,12 +32,12 @@ using Framebuffer = typename AppContext::Framebuffer;
 using Clock       = typename AppContext::Clock;
 
 constexpr std::array<std::uint8_t, font16x8::kGlyphHeight> kArrowUpGlyph{
-        0b00011000, 0b00111100, 0b01111110, 0b11111111, 0b00011000, 0b00011000, 0b00011000, 0b00011000,
-        0b00011000, 0b00011000, 0b00011000, 0b00011000, 0b00011000, 0b00011000, 0b00000000, 0b00000000};
+        0b00010000, 0b00111000, 0b01111100, 0b11111110, 0b00010000, 0b00010000, 0b00010000, 0b00010000,
+        0b00010000, 0b00010000, 0b00010000, 0b00010000, 0b00010000, 0b00010000, 0b00000000, 0b00000000};
 
 constexpr std::array<std::uint8_t, font16x8::kGlyphHeight> kArrowDownGlyph{
-        0b00000000, 0b00000000, 0b00011000, 0b00011000, 0b00011000, 0b00011000, 0b00011000, 0b00011000,
-        0b00011000, 0b00011000, 0b00011000, 0b11111111, 0b01111110, 0b00111100, 0b00011000, 0b00000000};
+        0b00000000, 0b00000000, 0b00010000, 0b00010000, 0b00010000, 0b00010000, 0b00010000, 0b00010000,
+        0b00010000, 0b00010000, 0b00010000, 0b11111110, 0b01111100, 0b00111000, 0b00010000, 0b00000000};
 
 struct TimeSnapshot {
     bool          hasWallTime   = false;
@@ -130,12 +131,14 @@ private:
         if (!notifications.empty() && (upPressed || downPressed)) {
             const std::size_t count       = notifications.size();
             lastNotificationInteractionMs = clock.millis();
-            navPressed                    = true;
             if (count > 1) {
-                if (upPressed)
-                    selectedNotification = (selectedNotification + count - 1) % count;
-                else if (downPressed)
-                    selectedNotification = (selectedNotification + 1) % count;
+                if (upPressed && selectedNotification > 0) {
+                    selectedNotification--;
+                    navPressed = true;
+                } else if (downPressed && selectedNotification < count - 1) {
+                    selectedNotification++;
+                    navPressed = true;
+                }
             }
         }
 
@@ -444,6 +447,18 @@ private:
 
         framebuffer.frameReady();
 
+        if (auto* battery = context.battery(); battery && battery->hasData()) {
+            const float percentage = battery->percentage();
+            if (std::isfinite(percentage) && percentage >= 0.0f) {
+                char pct[8];
+                std::snprintf(pct, sizeof(pct), "%.0f%%", static_cast<double>(percentage));
+                const int pctWidth = font16x8::measureText(pct, 1, 1);
+                const int pctX     = framebuffer.width() - pctWidth - 4;
+                const int pctY     = 4;
+                font16x8::drawText(framebuffer, pctX, pctY, pct, 1, true, 1);
+            }
+        }
+
         const int scaleTime      = 4;
         const int scaleSeconds   = 2;
         const int scaleSmall     = 1;
@@ -500,7 +515,9 @@ private:
                     const int arrowsTotalWide = arrowWidth * 2 + arrowSpacing;
                     const int arrowX          = counterX + (counterWidth - arrowsTotalWide) / 2;
                     const int arrowY          = cardMarginTop + cardPadding + textLineHeight + 1;
+                    if (selectedNotification > 0)
                         drawArrow(framebuffer, arrowX, arrowY, true, scaleSmall);
+                    if (selectedNotification < notifications.size() - 1)
                         drawArrow(framebuffer, arrowX + arrowWidth + arrowSpacing, arrowY, false, scaleSmall);
                     const int arrowHeight = font16x8::kGlyphHeight * scaleSmall;
                     cardHeight            = std::max(cardHeight, arrowY + arrowHeight - cardMarginTop);
@@ -524,28 +541,13 @@ private:
                 const int summaryX     = (framebuffer.width() - summaryWidth) / 2;
                 const int summaryY     = cardMarginTop;
                 font16x8::drawText(framebuffer, summaryX, summaryY, summary, scaleSmall, true, 1);
-
-                if (notifications.size() > 1) {
-                    const int arrowWidth      = font16x8::kGlyphWidth * scaleSmall;
-                    const int arrowSpacing    = std::max(1, scaleSmall);
-                    const int arrowsTotalWide = arrowWidth * 2 + arrowSpacing;
-                    const int arrowX          = (framebuffer.width() - arrowsTotalWide) / 2;
-                    const int arrowY          = summaryY + textLineHeight + 1;
-                    drawArrow(framebuffer, arrowX, arrowY, true, scaleSmall);
-                    drawArrow(framebuffer, arrowX + arrowWidth + arrowSpacing, arrowY, false, scaleSmall);
-                    const int arrowHeight = font16x8::kGlyphHeight * scaleSmall;
-                    cardHeight            = std::max(cardHeight, arrowY + arrowHeight - cardMarginTop);
-                }
             }
         }
 
         const int defaultTimeY = (framebuffer.height() - font16x8::kGlyphHeight * scaleTime) / 2 - 8;
-        int       timeY        = defaultTimeY;
-        if (cardHeight > 0)
-            timeY = cardMarginTop + cardHeight + 16;
         const int minTimeY     = (cardHeight > 0) ? (cardMarginTop + cardHeight + 12) : 16;
         const int maxTimeY     = std::max(minTimeY, framebuffer.height() - font16x8::kGlyphHeight * scaleTime - 48);
-        timeY              = std::clamp(timeY, minTimeY, maxTimeY);
+        const int timeY        = std::clamp(defaultTimeY, minTimeY, maxTimeY);
 
         char hoursMinutes[6];
         std::snprintf(hoursMinutes, sizeof(hoursMinutes), "%02d:%02d", snap.hour24, snap.minute);
@@ -562,24 +564,18 @@ private:
         const std::string dateLine = formatDate(snap);
         drawCenteredText(framebuffer, timeY + font16x8::kGlyphHeight * scaleTime + 16, dateLine, scaleSmall, 1);
 
-        const std::string instruction      = holdActive ? "KEEP HOLDING A+SELECT" : "HOLD A+SELECT";
+        const std::string instruction      = "HOLD A+SELECT TO UNLOCK";
         const int         instructionWidth = font16x8::measureText(instruction, scaleSmall, 1);
-        const int         barHeight        = 14;
-        const int         barY             = framebuffer.height() - 24;
-        const int         textY            = barY + (barHeight - textLineHeight) / 2;
-        const int         textX            = 8;
-        font16x8::drawText(framebuffer, textX, textY, instruction, scaleSmall, true, 1);
+        const int         barHeight        = 18;
+        const int         barY             = framebuffer.height() - 30;
+        const int         textY            = barY + (barHeight - textLineHeight) / 2 + 1;
+        drawCenteredText(framebuffer, textY, instruction, scaleSmall, 1);
 
-        int barX     = textX + instructionWidth + 12;
-        int barWidth = framebuffer.width() - barX - 8;
-        if (barWidth < 40) {
-            barWidth = 40;
-            barX     = std::min(barX, framebuffer.width() - barWidth - 8);
-        }
-
-        drawRectOutline(framebuffer, barX, barY, barWidth, barHeight);
+        const int barWidth = framebuffer.width() - 64;
+        const int barX     = 32;
 
         if (holdActive || holdProgressMs > 0) {
+            drawRectOutline(framebuffer, barX, barY, barWidth, barHeight);
             const int   innerWidth  = barWidth - 2;
             const int   innerHeight = barHeight - 2;
             const float ratio       = std::clamp(holdProgressMs / static_cast<float>(kUnlockHoldMs), 0.0f, 1.0f);

Firmware/sdk/backend_interface/include/cardboy/sdk/services.hpp

@@ -39,6 +39,7 @@ public:
     [[nodiscard]] virtual float voltage() const { return 0.0f; }
     [[nodiscard]] virtual float charge() const { return 0.0f; }
     [[nodiscard]] virtual float current() const { return 0.0f; }
+    [[nodiscard]] virtual float percentage() const { return 0.0f; }
 };
 
 class IStorage {

Firmware/sdk/backends/desktop/include/cardboy/backend/desktop_backend.hpp

@@ -40,7 +40,6 @@ public:
 class DesktopBattery final : public cardboy::sdk::IBatteryMonitor {
 public:
     [[nodiscard]] bool hasData() const override { return false; }
-};
 
     class DesktopStorage final : public cardboy::sdk::IStorage {
     public:

Firmware/sdk/core/include/cardboy/gfx/font16x8.hpp

@@ -14,8 +14,6 @@ constexpr unsigned char kFallbackChar = '?';
 
 inline unsigned char normalizeChar(char ch) {
     unsigned char uc = static_cast<unsigned char>(ch);
-    if (uc >= 'a' && uc <= 'z')
-        uc = static_cast<unsigned char>(std::toupper(static_cast<unsigned char>(uc)));
     if (!std::isprint(static_cast<unsigned char>(uc)))
         return kFallbackChar;
     return uc;

Firmware/sdk/core/include/cardboy/sdk/status_bar.hpp

@@ -56,9 +56,6 @@ private:
                 fb.drawPixel(x, y, true);
         }
 
-        for (int x = 0; x < width; ++x)
-            fb.drawPixel(x, 0, false);
-
         const int textY            = 1;
         const int bottomSeparatorY = textY + font16x8::kGlyphHeight + 1;
         if (bottomSeparatorY < fillHeight) {

Firmware/sdk/core/src/status_bar.cpp

@@ -52,14 +52,10 @@ std::string StatusBar::prepareRightText() const {
     if (_services->battery && _services->battery->hasData()) {
         const float current    = _services->battery->current();
         const float chargeMah  = _services->battery->charge();
-        const float fallbackV = _services->battery->voltage();
+        const float percentage = _services->battery->percentage();
         char        buf[64];
-        if (std::isfinite(current) && std::isfinite(chargeMah)) {
-            std::snprintf(buf, sizeof(buf), "cur %.2fmA chr %.2fmAh", static_cast<double>(current),
-                          static_cast<double>(chargeMah));
-        } else {
-            std::snprintf(buf, sizeof(buf), "vol %.2fV", static_cast<double>(fallbackV));
-        }
+        std::snprintf(buf, sizeof(buf), "%.2fmA %.2fmAh %.0f%%", static_cast<double>(current),
+                      static_cast<double>(chargeMah), static_cast<double>(percentage));
         right.assign(buf);
     }