feat: system-info dialog as a table + PN5180 reader firmware

Renders the information dialog as a property/value table instead of a
preformatted text blob and adds the PN5180 RFID-reader firmware version.
The version is read once from the reader EEPROM during init, cached, and
exposed through /info (hardware.readerFirmware) for RFID_READER_TYPE_RUNTIME
builds. New systeminfo label keys added to all locales.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: fhirschmann

README.md

@@ -88,6 +88,7 @@ neon logo that doubles as the SVG favicon ([`8d9725c`](../../commit/8d9725c)):
 | Control buttons (single-track Repeat toggle, Sleep-Timer with dropdown and live remaining countdown, Night Mode/Dimming) directly in the control tab of the web interface | [`6918d36`](../../commit/6918d36) |
 | Bluetooth-mode dropdown in the control tab (Normal / Speaker / Headphones), mirroring the sleep-timer dropdown; selecting a mode restarts the device into it and the button highlights the active BT mode | [`b9f1e5c`](../../commit/b9f1e5c) |
 | Mobile-optimized control tab: the hard-to-drag volume slider is replaced by full-width louder/quieter (+ EQ) buttons on phones, and the control header stacks so the legend stays on one line with the action buttons below | [`f10bc93`](../../commit/f10bc93) |
+| System-information dialog rendered as a clean property/value table (instead of preformatted text) and extended with the PN5180 RFID-reader firmware version (read once from the reader's EEPROM at init and exposed via `/info`) | [`56f5fda`](../../commit/56f5fda) |
 
 #### Feature highlights
 

html/locales/de.json

@@ -595,6 +595,25 @@
 		"seconds": "Sekunden"
 	},
 	"systeminfo": {
+		"lblSoftware": "Software",
+		"lblGit": "Git-Revision",
+		"lblEnv": "Arduino / ESP-IDF",
+		"lblChip": "Chip",
+		"lblCpu": "CPU-Takt",
+		"lblReader": "RFID-Reader-Firmware",
+		"lblHeap": "Freier Heap",
+		"lblHeapBlock": "Größter freier Block",
+		"lblPsram": "Freies PSRAM",
+		"lblIp": "IP-Adresse",
+		"lblMac": "MAC-Adresse",
+		"lblRssi": "WLAN-Signal",
+		"lblSince": "In Betrieb seit",
+		"lblPlayTotal": "Gesamte Spielzeit",
+		"lblPlaySince": "Spielzeit seit Start",
+		"lblSd": "SD-Karte",
+		"lblVoltage": "Akkuspannung",
+		"lblCharge": "Ladezustand",
+		"lblHall": "Hall-Sensor",
 		"title": "Information",
 		"softwareversion": "ESPuino {{softwareversion}}",
 		"gitversion": "ESPuino {{gitversion}}",

html/locales/en.json

@@ -597,6 +597,25 @@
 		"seconds": "seconds"
 	},
 	"systeminfo": {
+		"lblSoftware": "Software",
+		"lblGit": "Git revision",
+		"lblEnv": "Arduino / ESP-IDF",
+		"lblChip": "Chip",
+		"lblCpu": "CPU clock",
+		"lblReader": "RFID reader firmware",
+		"lblHeap": "Free heap",
+		"lblHeapBlock": "Largest free block",
+		"lblPsram": "Free PSRAM",
+		"lblIp": "IP address",
+		"lblMac": "MAC address",
+		"lblRssi": "WiFi signal",
+		"lblSince": "In service since",
+		"lblPlayTotal": "Total playtime",
+		"lblPlaySince": "Playtime since start",
+		"lblSd": "SD card",
+		"lblVoltage": "Battery voltage",
+		"lblCharge": "Charge level",
+		"lblHall": "Hall sensor",
 		"title": "Information",
 		"softwareversion": "ESPuino {{softwareversion}}",
 		"gitversion": "ESPuino {{gitversion}}",

html/locales/fr.json

@@ -584,6 +584,25 @@
 		"seconds": "secondes"
 	},
 	"systeminfo": {
+		"lblSoftware": "Logiciel",
+		"lblGit": "Révision Git",
+		"lblEnv": "Arduino / ESP-IDF",
+		"lblChip": "Puce",
+		"lblCpu": "Fréquence CPU",
+		"lblReader": "Firmware du lecteur RFID",
+		"lblHeap": "Mémoire libre",
+		"lblHeapBlock": "Plus grand bloc libre",
+		"lblPsram": "PSRAM libre",
+		"lblIp": "Adresse IP",
+		"lblMac": "Adresse MAC",
+		"lblRssi": "Signal WiFi",
+		"lblSince": "En service depuis",
+		"lblPlayTotal": "Temps de lecture total",
+		"lblPlaySince": "Lecture depuis le démarrage",
+		"lblSd": "Carte SD",
+		"lblVoltage": "Tension batterie",
+		"lblCharge": "Niveau de charge",
+		"lblHall": "Capteur Hall",
 		"title": "Informations",
 		"softwareversion": "ESPuino {{softwareversion}}",
 		"gitversion": "ESPuino {{gitversion}}",

html/management.html

@@ -1056,6 +1056,25 @@
 			font-family: "Share Tech Mono", monospace;
 		}
 
+		/* system information table */
+		.cp-info-table th {
+			white-space: nowrap;
+			width: 1%;
+			padding-right: 18px;
+			color: var(--cp-cyan);
+			font-weight: 600;
+		}
+
+		.cp-info-table td {
+			word-break: break-word;
+			color: var(--cp-text, #cfe9ff);
+		}
+
+		.cp-info-table tr+tr th,
+		.cp-info-table tr+tr td {
+			border-top: 1px solid var(--cp-border);
+		}
+
 		/* ---------- toasts ---------- */
 		.toast {
 			background: var(--cp-panel);
@@ -1441,7 +1460,7 @@
 				<div class="modal-header">
 					<h5 class="modal-title" data-i18n="infos"></h5>
 				</div>
-				<div class="modal-body" style="white-space: pre" id="modalInfoContent">
+				<div class="modal-body" id="modalInfoContent">
 				</div>
 				<div class="modal-footer">
 					<button type="button" id="modalInfoRefresh" class="btn btn-primary me-auto"><i
@@ -5442,76 +5461,38 @@ <h5 class="modal-title" data-i18n="tools.nvs.erase.title"></h5>
 		async function fetchInfo() {
 			let info = await (await fetch("http://" + host + "/info")).json();
 			console.log(info);
-			let content = i18next.t("systeminfo.softwareversion", {
-				softwareversion: info.software.version
-			}) + "<br>";
-			content += i18next.t("systeminfo.gitversion", {
-				gitversion: info.software.git
-			}) + "<br>";
-			content += i18next.t("systeminfo.arduinoversion", {
-				arduinoversion: info.software.arduino,
-				idfversion: info.software.idf
-			}) + "<br>";
-			content += i18next.t("systeminfo.hardware", {
-				hwmodel: info.hardware.model,
-				hwrevision: info.hardware.revision,
-				hwfreq: info.hardware.freq
-			}) + "<br>";
-			content += i18next.t("systeminfo.freeheap", {
-				freeheap: info.memory.freeHeap
-			}) + "<br>";
-			content += i18next.t("systeminfo.largestfreeblock", {
-				largestfreeblock: info.memory.largestFreeBlock
-			}) + "<br>";
-			if (info.memory.freePSRam) {
-				content += i18next.t("systeminfo.freepsram", {
-					freepsram: info.memory.freePSRam
-				}) + "<br>";
-			}
-			content += i18next.t("systeminfo.currentip", {
-				currentip: info.wifi.ip
-			}) + "<br>";
-			content += i18next.t("systeminfo.macAddress", {
-				macAddress: info.wifi.macAddress
-			}) + "<br>";
-			content += i18next.t("systeminfo.rssi", {
-				rssi: info.wifi.rssi
-			}) + "<br>";
-			content += i18next.t("systeminfo.audiotimetotal", {
-				firststart: secondsToDate(info.audio.firstStart),
-				audiotimetotal: timestampToDuration(info.audio.playtimeTotal, false)
-			}) + "<br>";
-			if (info.audio.playtimeSinceStart > 0) {
-				content += i18next.t("systeminfo.audiotimesincestart", {
-					audiotimesincestart: timestampToDuration(info.audio.playtimeSinceStart, true)
-				}) + "<br>";
-			}
-			if (info.sdcard && info.sdcard.size) {
-				content += i18next.t("systeminfo.sdcard", {
-					free: (info.sdcard.free / 1024).toFixed(1),
-					total: (info.sdcard.size / 1024).toFixed(1)
-				}) + "<br>";
-			}
+			const esc = (typeof escapeHtml === 'function') ? escapeHtml : function (x) { return x; };
+			const num = function (n) { return Number(n).toLocaleString(); };
+			const rows = [];
+			const add = function (labelKey, value) {
+				if (value === undefined || value === null || value === '') return;
+				rows.push('<tr><th scope="row">' + i18next.t(labelKey) + '</th><td>' + esc(String(value)) + '</td></tr>');
+			};
+
+			add('systeminfo.lblSoftware', info.software.version);
+			add('systeminfo.lblGit', info.software.git);
+			add('systeminfo.lblEnv', info.software.arduino + ' / ' + info.software.idf);
+			add('systeminfo.lblChip', info.hardware.model + ' (rev ' + info.hardware.revision + ')');
+			add('systeminfo.lblCpu', info.hardware.freq + ' MHz');
+			if (info.hardware.readerFirmware) add('systeminfo.lblReader', 'PN5180 ' + info.hardware.readerFirmware);
+			add('systeminfo.lblHeap', num(info.memory.freeHeap) + ' B');
+			add('systeminfo.lblHeapBlock', num(info.memory.largestFreeBlock) + ' B');
+			if (info.memory.freePSRam) add('systeminfo.lblPsram', num(info.memory.freePSRam) + ' B');
+			add('systeminfo.lblIp', info.wifi.ip);
+			add('systeminfo.lblMac', info.wifi.macAddress);
+			add('systeminfo.lblRssi', info.wifi.rssi + ' dBm');
+			add('systeminfo.lblSince', secondsToDate(info.audio.firstStart));
+			add('systeminfo.lblPlayTotal', timestampToDuration(info.audio.playtimeTotal, false));
+			if (info.audio.playtimeSinceStart > 0) add('systeminfo.lblPlaySince', timestampToDuration(info.audio.playtimeSinceStart, true));
+			if (info.sdcard && info.sdcard.size) add('systeminfo.lblSd', (info.sdcard.free / 1024).toFixed(1) + ' / ' + (info.sdcard.size / 1024).toFixed(1) + ' GB');
 			if (info.battery) {
-				content += i18next.t("systeminfo.currvoltage", {
-					currvoltage: info.battery.currVoltage.toFixed(2)
-				}) + "<br>";
-				if (info.battery.chargelevel) {
-					content += i18next.t("systeminfo.chargelevel", {
-						chargelevel: info.battery.chargeLevel.toFixed(1)
-					}) + "<br>";
-				}
+				add('systeminfo.lblVoltage', info.battery.currVoltage.toFixed(2) + ' V');
+				if (info.battery.chargeLevel) add('systeminfo.lblCharge', info.battery.chargeLevel.toFixed(1) + ' %');
 			}
-			if (info.hallsensor) {
-				content += i18next.t("systeminfo.hallsensor", {
-					hsnullfieldvalue: info.hallsensor.nullFieldValue,
-					hsactual: info.hallsensor.actual,
-					hsdiff: info.hallsensor.diff,
-					hslastwaitforstate: info.hallsensor.lastWaitState,
-					hswaited: info.hallsensor.lastWaitMS
-				}) + "<br>";
-			}
-			$('#modalInfoContent').html(content);
+			if (info.hallsensor) add('systeminfo.lblHall', 'null ' + info.hallsensor.nullFieldValue + ', actual ' + info.hallsensor.actual + ', diff ' + info.hallsensor.diff);
+
+			const html = '<table class="table table-sm table-borderless cp-info-table mb-0"><tbody>' + rows.join('') + '</tbody></table>';
+			$('#modalInfoContent').html(html);
 		}
 		var liveLogInterval;
 		async function fetchLog() {

src/Rfid.h

@@ -14,3 +14,4 @@ void Rfid_TaskResume(void);
 void Rfid_TaskReset(void);
 void Rfid_WakeupCheck(void);
 void Rfid_PreferenceLookupHandler(void);
+const char *Rfid_GetReaderFirmwareVersion(void); // PN5180 reader firmware version (RFID_READER_TYPE_RUNTIME builds only)

src/RfidPn5180.cpp

@@ -47,6 +47,7 @@ static void RfidPn5180_Task(void *parameter);
 uint8_t stateMachine = RFID_PN5180_STATE_INIT;
 static bool rfidTaskResetRequested = false;
 static byte lastValidcardId[cardIdSize];
+static char Rfid_ReaderFwVersion[12] = ""; // cached PN5180 firmware version (e.g. "4.0"), read once at init
 
 static bool Rfid_Pn5180LpcdEnabled(void) {
 	return gPrefsRfid.getBool("pn5180Lpcd", false);
@@ -188,10 +189,17 @@ void RfidPn5180_Task(void *parameter) {
 		if (RFID_PN5180_STATE_INIT == stateMachine) {
 			nfc14443.begin();
 			nfc14443.reset();
-			// show PN5180 reader version
-			// uint8_t firmwareVersion[2];
-			// nfc14443.readEEprom(FIRMWARE_VERSION, firmwareVersion, sizeof(firmwareVersion));
-			// Log_Printf(LOGLEVEL_DEBUG, "PN5180 firmware version=%d.%d", firmwareVersion[1], firmwareVersion[0]);
+			// read and cache the PN5180 reader firmware version once (for the info dialog).
+			// Only cache a plausible value: if no (or a non-PN5180) reader is connected the
+			// EEPROM read yields garbage (0x00/0xFF), which we reject so /info omits the field.
+			if (Rfid_ReaderFwVersion[0] == '\0') {
+				uint8_t firmwareVersion[2] = {0, 0};
+				if (nfc14443.readEEprom(FIRMWARE_VERSION, firmwareVersion, sizeof(firmwareVersion))
+					&& firmwareVersion[1] != 0x00 && firmwareVersion[1] != 0xFF) {
+					snprintf(Rfid_ReaderFwVersion, sizeof(Rfid_ReaderFwVersion), "%d.%d", firmwareVersion[1], firmwareVersion[0]);
+					Log_Printf(LOGLEVEL_DEBUG, "PN5180 firmware version=%s", Rfid_ReaderFwVersion);
+				}
+			}
 
 			// activate RF field
 			delay(4u);
@@ -537,4 +545,9 @@ void RfidPn5180_WakeupCheck(void) {
 	}
 	nfc14443.end();
 }
+
+// Returns the cached PN5180 reader firmware version (e.g. "4.0"), or "" if not yet read.
+const char *Rfid_GetReaderFirmwareVersion(void) {
+	return Rfid_ReaderFwVersion;
+}
 #endif

src/Web.cpp

@@ -1712,6 +1712,13 @@ void handleGetInfo(AsyncWebServerRequest *request) {
 		hardwareObj["model"] = String(ESP.getChipModel());
 		hardwareObj["revision"] = ESP.getChipRevision();
 		hardwareObj["freq"] = ESP.getCpuFreqMHz();
+#if defined(RFID_READER_TYPE_RUNTIME)
+		// PN5180 reader firmware version (empty until the reader has been initialised once)
+		const char *readerFw = Rfid_GetReaderFirmwareVersion();
+		if (readerFw && readerFw[0] != '\0') {
+			hardwareObj["readerFirmware"] = String(readerFw);
+		}
+#endif
 	}
 	// memory
 	if ((section == "") || (section == "memory")) {