Просмотр исходного кода

fix: hide battery voltage range text for PMU-backed web stats

Jared Dohrman 3 месяцев назад
Родитель
Сommit
5c50295552
2 измененных файлов с 3 добавлено и 2 удалено
  1. 1 1
      docs/web-panel.md
  2. 2 1
      src/helpers/web/WebPanelServer.cpp

+ 1 - 1
docs/web-panel.md

@@ -233,7 +233,7 @@ For boards that expose extra telemetry, the optional `Environment` summary card
 
 Metrics with no current value are hidden rather than showing placeholder rows, so the cards vary by board and by current sensor state.
 
-The `Core` battery meter prefers a board-reported battery percentage when the target exposes one. Otherwise it scales the displayed percentage from the board's configured battery voltage range rather than assuming a fixed single-cell `3000-4200 mV` pack.
+The `Core` battery meter prefers a board-reported battery percentage when the target exposes one. On those boards, the meter detail shows the live battery millivolt reading only. Otherwise it scales the displayed percentage from the board's configured battery voltage range and shows that range in the detail text rather than assuming a fixed single-cell `3000-4200 mV` pack.
 
 The trend graphs load sequentially rather than as one large payload:
 

+ 2 - 1
src/helpers/web/WebPanelServer.cpp

@@ -1472,7 +1472,8 @@ const char kWebPanelAppHtml[] PROGMEM = R"HTML(
           : pctRange(core.battery_mv, batteryMin, batteryMax);
       const queuePct = pctRange(core.queue_len, 0, 12);
       const errorsPct = core.errors > 0 ? 100 : 0;
-      const batteryDetail = batteryMin < batteryMax
+      const usesBoardBatteryPercent = Number.isFinite(core.battery_pct) && core.battery_pct >= 0;
+      const batteryDetail = !usesBoardBatteryPercent && batteryMin < batteryMax
           ? (core.battery_mv || 0) + " mV (" + batteryMin + "-" + batteryMax + " mV)"
           : (core.battery_mv || 0) + " mV";
       return `<section class="hud-card">