From 352e1e322c6ceb7dcc81f84d859853109e5d41fa Mon Sep 17 00:00:00 2001 From: dborup Date: Tue, 21 Jul 2026 07:56:07 +0200 Subject: [PATCH] feat: reorder Wardriving tab and collapse Top Senders/Coverage to top 10 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signal Quality Trends now renders right after the message-volume chart, above Top Senders and Coverage by Observer (previously it was lower, after Sessions/Entry Points). Top Senders and Coverage by Observer collapse to the top 10 rows by default, with a "Show all N" / "Show fewer" toggle — same pattern already used for node neighbor lists (nodes.js). Coverage's "% of Observations" stays relative to ALL observers, not just the shown top-10 slice, so the percentages don't lie when collapsed. Co-Authored-By: Claude Sonnet 5 --- public/analytics.js | 70 +++++++++++++++++++++++++------- test-analytics-wardriving-tab.js | 30 ++++++++++++++ 2 files changed, 85 insertions(+), 15 deletions(-) diff --git a/public/analytics.js b/public/analytics.js index cbf36502..0b9160a9 100644 --- a/public/analytics.js +++ b/public/analytics.js @@ -5558,17 +5558,29 @@ function destroy() { _stopRolesRefresh(); _stopScopesRefresh(); _stopForeignTraf ''; } - function sendersHtml(senders, totalMessages) { + var TOP_N_LIMIT = 10; + + // Shared "Show all N / Show fewer" toggle trigger for any collapsible + // table below. The click handler itself is wired generically by + // wireExpandToggle, keyed on the data-wd-toggle attribute. + function topNToggleHtml(total, expanded, noun) { + if (total <= TOP_N_LIMIT) return ''; + var label = expanded ? 'Show fewer' : 'Show all ' + total.toLocaleString() + ' ' + noun; + return '
'; + } + + function sendersHtml(senders, totalMessages, expanded) { if (!senders || senders.length === 0) { return '

No wardriving messages in this window.

'; } - var rows = senders.map(function(s) { + var shown = expanded ? senders : senders.slice(0, TOP_N_LIMIT); + var rows = shown.map(function(s) { return '' + senderTriggerHtml(s.sender) + '' + s.count.toLocaleString() + '' + pct(s.count, totalMessages) + ''; }).join(''); return '' + '' + '' + rows + '' + - '
SenderMessages% of Total
'; + '' + topNToggleHtml(senders.length, expanded, 'senders'); } // Entry Points — resolve raw path[0] hash prefixes to repeater names, @@ -5616,12 +5628,15 @@ function destroy() { _stopRolesRefresh(); _stopScopesRefresh(); _stopForeignTraf } } - function observersHtml(observers) { + function observersHtml(observers, expanded) { if (!observers || observers.length === 0) { return '

No observer has heard wardriving traffic in this window.

'; } + // % of Observations stays relative to ALL observers, not just the + // shown top-N slice, so the numbers stay meaningful when collapsed. var totalObsCount = observers.reduce(function(sum, o) { return sum + o.observationCount; }, 0); - var rows = observers.map(function(o) { + var shown = expanded ? observers : observers.slice(0, TOP_N_LIMIT); + var rows = shown.map(function(o) { var loc = (o.lat != null && o.lon != null) ? (o.lat.toFixed(2) + ', ' + o.lon.toFixed(2)) : '—'; return '' + esc(o.observerName) + '' + '' + esc(o.iata || '—') + '' + @@ -5633,7 +5648,7 @@ function destroy() { _stopRolesRefresh(); _stopScopesRefresh(); _stopForeignTraf return '' + '' + '' + rows + '' + - '
ObserverRegionLat, LonObservations% of ObservationsDistinct Messages
'; + '' + topNToggleHtml(observers.length, expanded, 'observers'); } function mapLinkHtml(lat, lon) { @@ -5886,6 +5901,27 @@ function destroy() { _stopRolesRefresh(); _stopScopesRefresh(); _stopForeignTraf }); } + // Generic "Show all / Show fewer" wiring for a collapsible section — + // re-renders just that section's content on click and rewires the + // (freshly-created) toggle button, since innerHTML replacement drops + // the old node's listener. + function wireExpandToggle(containerId, renderFn) { + var expanded = false; + function attach() { + var container = document.getElementById(containerId); + if (!container) return; + var btn = container.querySelector('[data-wd-toggle]'); + if (btn) { + btn.addEventListener('click', function() { + expanded = !expanded; + container.innerHTML = renderFn(expanded); + attach(); + }); + } + } + attach(); + } + async function load(w) { var body; try { @@ -5894,9 +5930,15 @@ function destroy() { _stopRolesRefresh(); _stopScopesRefresh(); _stopForeignTraf body = '
' + cardsHtml(d) + '
' + '
' + chartHtml(d.timeSeries) + '
' + - '

Top Senders

' + + '

Signal Quality Trends

' + + '

Average SNR and RSSI across every observation of wardriving traffic in each time bucket — a rough proxy for link quality, not tied to any one observer.

' + + '
' + + '
Avg SNR (dB)
' + signalChartHtml(d.signalTimeSeries, 'avgSnr', 'var(--accent)', 'Average SNR over time') + '
' + + '
Avg RSSI (dBm)
' + signalChartHtml(d.signalTimeSeries, 'avgRssi', 'var(--warning, #f39c12)', 'Average RSSI over time') + '
' + + '
' + + '

Top Senders

' + '

Who\'s actively wardriving in this window, by message count.

' + - '
' + sendersHtml(d.topSenders, d.totalMessages) + '
' + + '
' + sendersHtml(d.topSenders, d.totalMessages, false) + '
' + '

Sessions

' + '

Each sender\'s messages grouped into distinct runs — a gap of more than 15 minutes starts a new session.

' + '
' + sessionsHtml(d.sessions) + '
' + @@ -5905,13 +5947,7 @@ function destroy() { _stopRolesRefresh(); _stopScopesRefresh(); _stopForeignTraf '
' + entryHtml + '
' + '

Coverage by Observer

' + '

Which observer stations actually heard wardriving traffic — observers sit at fixed, known locations, so this is the reliable half of "how far did it reach."

' + - '
' + observersHtml(d.observers) + '
' + - '

Signal Quality Trends

' + - '

Average SNR and RSSI across every observation of wardriving traffic in each time bucket — a rough proxy for link quality, not tied to any one observer.

' + - '
' + - '
Avg SNR (dB)
' + signalChartHtml(d.signalTimeSeries, 'avgSnr', 'var(--accent)', 'Average SNR over time') + '
' + - '
Avg RSSI (dBm)
' + signalChartHtml(d.signalTimeSeries, 'avgRssi', 'var(--warning, #f39c12)', 'Average RSSI over time') + '
' + - '
' + + '
' + observersHtml(d.observers, false) + '
' + '

GPS Sharing

' + '

Senders whose client appended their own position after the standard token — a deliberate choice by that sender to share their location, not something inferred from the anonymous token itself.

' + '
' + gpsSharesHtml(d.gpsShares) + '
'; @@ -5932,6 +5968,10 @@ function destroy() { _stopRolesRefresh(); _stopScopesRefresh(); _stopForeignTraf body; attachWindowButtons(); attachSenderDrilldown(); + if (d) { + wireExpandToggle('wardrivingSenders', function(exp) { return sendersHtml(d.topSenders, d.totalMessages, exp); }); + wireExpandToggle('wardrivingObservers', function(exp) { return observersHtml(d.observers, exp); }); + } } await load(selectedWindow); diff --git a/test-analytics-wardriving-tab.js b/test-analytics-wardriving-tab.js index f8499820..b8717089 100644 --- a/test-analytics-wardriving-tab.js +++ b/test-analytics-wardriving-tab.js @@ -221,6 +221,36 @@ function makeApiStub(wardrivingResp, resolveHopsResp) { assert.ok(el.innerHTML.includes('66.7%'), 'Alice row should show 66.7% of total messages'); }); + await testAsync('Signal Quality Trends renders above Top Senders (moved up per layout request)', async () => { + const ctx = makeAnalyticsSandbox(makeApiStub(makeWardrivingResponse())); + const el = fakeEl(); + await ctx.window._analyticsRenderWardrivingTab(el); + const idxSignal = el.innerHTML.indexOf('Signal Quality Trends'); + const idxSenders = el.innerHTML.indexOf('>Top Senders<'); + assert.ok(idxSignal > -1 && idxSenders > -1, 'both sections should render'); + assert.ok(idxSignal < idxSenders, 'Signal Quality Trends should render before Top Senders'); + }); + + await testAsync('Top Senders and Coverage by Observer collapse to top 10 with a "Show all" toggle', async () => { + const manySenders = []; + for (let i = 0; i < 15; i++) manySenders.push({ sender: 'Sender' + i, count: 15 - i }); + const manyObservers = []; + for (let i = 0; i < 12; i++) manyObservers.push({ observerId: String(i), observerName: 'Observer' + i, observationCount: 12 - i, messageCount: 1 }); + const ctx = makeAnalyticsSandbox(makeApiStub(makeWardrivingResponse({ topSenders: manySenders, observers: manyObservers }))); + const el = fakeEl(); + await ctx.window._analyticsRenderWardrivingTab(el); + + const sendersSection = el.innerHTML.slice(el.innerHTML.indexOf('id="wardrivingSenders"'), el.innerHTML.indexOf('id="wardrivingSessions"')); + assert.ok(sendersSection.includes('Sender9'), 'the 10th sender (index 9) should be shown'); + assert.ok(!sendersSection.includes('Sender10'), 'the 11th sender should be collapsed away by default'); + assert.ok(sendersSection.includes('Show all 15 senders'), 'a "Show all" toggle should appear when there are more than 10 senders'); + + const observersSection = el.innerHTML.slice(el.innerHTML.indexOf('id="wardrivingObservers"'), el.innerHTML.indexOf('id="wardrivingGPSShares"')); + assert.ok(observersSection.includes('Observer9'), 'the 10th observer (index 9) should be shown'); + assert.ok(!observersSection.includes('Observer10'), 'the 11th observer should be collapsed away by default'); + assert.ok(observersSection.includes('Show all 12 observers'), 'a "Show all" toggle should appear when there are more than 10 observers'); + }); + await testAsync('Top Senders names are clickable drill-down triggers (whole-window, no since/until)', async () => { const ctx = makeAnalyticsSandbox(makeApiStub(makeWardrivingResponse())); const el = fakeEl();